Skip to content

[Bug]: ProtoByteBuddyUtils should use ByteBuddyUtils.getClassLoadingStrategy #38905

@cushon

Description

@cushon

What happened?

ProtoByteBuddyUtils uses ClassLoadingStrategy.Default.INJECTION. See #25578, which previously migrated some uses of that strategy.

ClassLoadingStrategy.Default.INJECTION uses sun.misc.Unsafe internally, which is deprecated (https://openjdk.org/jeps/471) and the latest versions of bytebuddy warn on using that strategy on newer JDK versions (https://github.com/raphw/byte-buddy/releases/tag/byte-buddy-1.18.9)

ProtoByteBuddyUtils should be updated to use ByteBuddyUtils.getClassLoadingStrategy, or equivalent logic if it can't depend directly on ByteBuddyUtils.getClassLoadingStrategy, instead of ClassLoadingStrategy.Default.INJECTION.

Issue Priority

Priority: 2 (default / most bugs should be filed as P2)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam YAML
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Infrastructure
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Prism Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions