Skip to content

Commit 1701e86

Browse files
committed
Use li-iceberg
1 parent ca0936b commit 1701e86

File tree

5 files changed

+24
-4
lines changed

5 files changed

+24
-4
lines changed

apps/spark-3.5/build.gradle

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ ext {
1818

1919
configurations.all {
2020
exclude group: 'org.apache.iceberg', module: 'iceberg-data'
21+
exclude group: 'com.linkedin.iceberg', module: 'iceberg-data'
2122
}
2223
dependencies {
2324
compileOnly (project(path: openhouseSparkRuntimeModule)) {
@@ -29,6 +30,7 @@ dependencies {
2930
implementation "org.apache.logging.log4j:log4j-slf4j-impl:${log4jVersion}"
3031
implementation(project(':libs:datalayout')) {
3132
exclude group: 'org.apache.iceberg', module: 'iceberg-spark-runtime-3.1_2.12'
33+
exclude group: 'com.linkedin.iceberg', module: 'iceberg-spark-runtime-3.1_2.12'
3234
}
3335
implementation ('org.apache.spark:spark-core_2.12:' + sparkVersion) {
3436
exclude group: 'io.netty'

build.gradle

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,22 @@ allprojects {
6969
force 'com.fasterxml.jackson.core:jackson-databind:2.13.4'
7070
force 'org.apache.orc:orc-core:1.8.3'
7171
force 'com.google.guava:guava:31.1-jre'
72+
73+
// Redirect Apache Iceberg 1.2.0 core modules to LinkedIn groupId/version without changing declarations/imports
74+
dependencySubstitution {
75+
all { sub ->
76+
def req = sub.requested
77+
if (req instanceof org.gradle.api.artifacts.component.ModuleComponentSelector && req.group == 'org.apache.iceberg') {
78+
def module = req.module
79+
def version = req.version
80+
boolean isIceberg120 = version != null && version == '1.2.0'
81+
boolean isAwsOrAzure = module != null && (module.startsWith('iceberg-aws') || module.startsWith('iceberg-azure'))
82+
if (isIceberg120 && !isAwsOrAzure) {
83+
sub.useTarget("com.linkedin.iceberg:${module}:1.2.0.0-rc2")
84+
}
85+
}
86+
}
87+
}
7288
}
7389
}
7490
}

buildSrc/src/main/groovy/openhouse.apps-spark-common.gradle

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,6 @@ dependencies {
2222
implementation project(':services:common')
2323
implementation project(':cluster:storage')
2424

25-
implementation("org.apache.iceberg:iceberg-bundled-guava")
26-
implementation("org.apache.iceberg:iceberg-data")
27-
implementation("org.apache.iceberg:iceberg-core")
28-
implementation("org.apache.iceberg:iceberg-common")
2925

3026
implementation ('org.apache.hadoop:hadoop-common:2.10.0') {
3127
exclude group: 'io.netty'
@@ -74,6 +70,10 @@ shadowJar {
7470
// this dependency will be provided at runtime
7571
exclude(dependency('org.apache.iceberg:iceberg-spark3-runtime::'))
7672

73+
// mirror excludes for com.linkedin.iceberg groupId after dependency substitution
74+
exclude(dependency('com.linkedin.iceberg::'))
75+
exclude(dependency('com.linkedin.iceberg:iceberg-spark3-runtime::'))
76+
7777
// can possibly be removed with gradle8 and shadow upgrade
7878
// https://github.com/GradleUp/shadow/issues/877#issuecomment-2472922043
7979
exclude 'META-INF/versions/19/'

scripts/java/tools/dummytokens/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ mainClassName = 'com.linkedin.openhouse.tools.dummytokens.DummyTokenGenerator'
99
dependencies {
1010
implementation(project(':services:common')){
1111
exclude group: 'org.apache.iceberg'
12+
exclude group: 'com.linkedin.iceberg'
1213
exclude group: 'com.azure'
1314
}
1415
implementation 'commons-cli:commons-cli:1.5.0'

services/tables/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,6 @@ dependencies {
3737
testImplementation(testFixtures(project(':services:common')))
3838
testImplementation (project(':tables-test-fixtures:tables-test-fixtures_2.12')) {
3939
exclude group: 'org.apache.iceberg'
40+
exclude group: 'com.linkedin.iceberg'
4041
}
4142
}

0 commit comments

Comments
 (0)