Skip to content

Conversation

@zhangskz
Copy link
Member

…code using lazy feature resolution.

This should not impact current gencode from >=4.x.x which resolves features in static init.

Fixes #20599

PiperOrigin-RevId: 738467326

Cherry-pick of fafb0fd

…code using lazy feature resolution.

This should not impact current gencode from >=4.x.x which resolves features in static init.

Fixes #20599

PiperOrigin-RevId: 738467326
@zhangskz zhangskz requested review from a team as code owners March 19, 2025 18:17
@zhangskz zhangskz requested review from deannagarcia, mkruskal-google and shaod2 and removed request for a team, deannagarcia and shaod2 March 19, 2025 18:17
@zhangskz zhangskz enabled auto-merge (squash) March 19, 2025 18:18
@zhangskz zhangskz merged commit ce59907 into 29.x Mar 19, 2025
179 checks passed
@zhangskz zhangskz deleted the cp-java-concurrency branch March 19, 2025 19:23
startSignal,
doneSignal,
// Static method invocation triggers static initialization.
() -> Assert.assertNotNull(UnittestProto.getDescriptor())));

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this UnittestProto generated by protoc 25.x?

Copy link
Member Author

@zhangskz zhangskz Mar 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes -- //java/core:v25ConcurrentDescriptorsTest uses UnittestProto from v25 gencode jars.

junit_tests(
name = "v25_core_tests_jar",
size = "small",
srcs = glob(
["src/test/java/**/*.java"],
exclude = [
# Depends on test protos or API changes added in v4.x.x (e.g. editions)
"src/test/java/com/google/protobuf/TextFormatTest.java",
"src/test/java/com/google/protobuf/DescriptorsTest.java",
"src/test/java/com/google/protobuf/DebugFormatTest.java",
"src/test/java/com/google/protobuf/CodedOutputStreamTest.java",
"src/test/java/com/google/protobuf/CodedInputStreamTest.java",
"src/test/java/com/google/protobuf/ProtobufToStringOutputTest.java",
"src/test/java/com/google/protobuf/FieldPresenceTest.java",
# Excluded in core_tests
"src/test/java/com/google/protobuf/DecodeUtf8Test.java",
"src/test/java/com/google/protobuf/IsValidUtf8Test.java",
"src/test/java/com/google/protobuf/TestUtil.java",
"src/test/java/com/google/protobuf/TestUtilLite.java",
"src/test/java/com/google/protobuf/RuntimeVersionTest.java",
],
),
test_prefix = "v25Jar",
deps = [
":core",
":v25_test_util_jar",
"//compatibility:v25_test_protos_jar",
"@protobuf_maven_dev//:com_google_guava_guava",
"@protobuf_maven_dev//:com_google_truth_truth",
"@protobuf_maven_dev//:junit_junit",
"@protobuf_maven_dev//:org_mockito_mockito_core",
],
)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants