Marton Greber has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23606 )

Change subject: KUDU-3662: Fix shading in kudu-replication
......................................................................

KUDU-3662: Fix shading in kudu-replication

Mark Flink core APIs (flink-streaming-java, flink-core,
flink-clients, flink-connector-base) as compileUnshaded (Maven
'provided' scope). These are supplied by the Flink cluster at
runtime and should not be bundled to avoid classpath conflicts.

Bundle flink-connector-kudu but exclude its kudu-client
dependency to avoid duplicates, using our shaded kudu-client
instead.

Add shadowJar exclusions for cluster-provided dependencies:
Hadoop classes, commons-cli, and logging bindings (slf4j,
logback, log4j).

Extend testImplementation from compileUnshaded to ensure test
code can compile against the Flink APIs.

This follows the standard Flink connector packaging pattern as
documented at:
https://nightlies.apache.org/flink/flink-docs-release-1.20/docs/dev/
configuration/overview/

Change-Id: I57660c7c97fd440309f69f5d5e0d028aa88042dc
Reviewed-on: http://gerrit.cloudera.org:8080/23606
Reviewed-by: Zoltan Chovan <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Marton Greber <[email protected]>
---
M java/kudu-replication/build.gradle
1 file changed, 30 insertions(+), 6 deletions(-)

Approvals:
  Zoltan Chovan: Looks good to me, but someone else must approve
  Abhishek Chennaka: Looks good to me, approved
  Alexey Serbin: Looks good to me, approved
  Marton Greber: Verified

--
To view, visit http://gerrit.cloudera.org:8080/23606
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I57660c7c97fd440309f69f5d5e0d028aa88042dc
Gerrit-Change-Number: 23606
Gerrit-PatchSet: 4
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to