Skip to content

[improve][io] Align the OpenMLDB module's forced protobuf with the platform version #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
david-streamlio wants to merge 1 commit into
base: master
Choose a base branch
from
+8 −5
Open

[improve][io] Align the OpenMLDB module's forced protobuf with the platform version #106

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 8 additions & 5 deletions jdbc/openmldb/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,16 +30,19 @@ plugins {
// - Curator: the SDK calls NodeCache.getListenable() expecting the 4.x signature (returning
// ListenerContainer); Curator 5.x changed it to return Listenable -> NoSuchMethodError on
// connect under the platform's 5.7.1.
// - Protobuf: the SDK's generated classes (protoc 3.16) call makeExtensionsImmutable(), removed
// in protobuf-java 3.22 -> NoSuchMethodError on insert under the platform's 3.25.5. 3.21.12 is
// the newest runtime that still has it. (NARs never bundle protobuf — the Pulsar runtime
// provides it — so this only affects compile/test classpaths.)
// - Protobuf: the SDK's generated classes (protoc 3.16) call makeExtensionsImmutable(), which
// protobuf-java 4.x removed (the 3.25.x line still has it). Without the force, the TEST
// classpath resolves protobuf 4.x by conflict resolution — io.kubernetes:client-java, a
// transitive of the test-only pulsar-functions-instance, requests it — and the SDK throws
// NoSuchMethodError on insert. Force the platform's 3.25.5: the same line the Pulsar runtime
// provides to the NAR in production (NARs never bundle protobuf), so the test matches what
// the connector actually runs against.
configurations.all {
resolutionStrategy.force(
"org.apache.curator:curator-client:4.2.0",
"org.apache.curator:curator-framework:4.2.0",
"org.apache.curator:curator-recipes:4.2.0",
"com.google.protobuf:protobuf-java:3.21.12",
"com.google.protobuf:protobuf-java:3.25.5",
)
}

Expand Down
Loading