Skip to content
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.

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

[#2645]fix(build): Fix the bugs that can't publish Gravitino jars to maven repo #2646

Merged
merged 9 commits into from
Apr 10, 2024
4 changes: 4 additions & 0 deletions catalogs/catalog-hadoop/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -105,3 +105,7 @@ tasks.test {
init(this)
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
4 changes: 4 additions & 0 deletions catalogs/catalog-hive/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -171,3 +171,7 @@ tasks.test {
init(this)
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
4 changes: 4 additions & 0 deletions catalogs/catalog-jdbc-doris/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -86,3 +86,7 @@ tasks.test {
init(this)
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
4 changes: 4 additions & 0 deletions catalogs/catalog-jdbc-mysql/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -86,3 +86,7 @@ tasks.test {
init(this)
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
4 changes: 4 additions & 0 deletions catalogs/catalog-jdbc-postgresql/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -92,3 +92,7 @@ tasks.test {
init(this)
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
5 changes: 5 additions & 0 deletions catalogs/catalog-kafka/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,11 @@ tasks {
}
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}

// TODO. add test task later on.
tasks.test {
val skipUTs = project.hasProperty("skipTests")
if (skipUTs) {
Expand Down
4 changes: 4 additions & 0 deletions catalogs/catalog-lakehouse-iceberg/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -173,3 +173,7 @@ tasks.test {
tasks.clean {
delete("spark-warehouse")
}

tasks.getByName("generateMetadataFileForMavenJavaPublication") {
dependsOn("runtimeJars")
}
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@ public Transform unknown(int fieldId, String sourceName, int sourceId, String tr
/**
* Transform assembled into gravitino.
*
* @param partitionSpec
* @param schema
* @param partitionSpec Iceberg partition spec.
* @param schema Iceberg schema.
* @return array of transforms for partition fields.
*/
public static Transform[] fromPartitionSpec(PartitionSpec partitionSpec, Schema schema) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ private NullOrdering toGravitino(NullOrder nullOrder) {
/**
* Convert Iceberg order to Gravitino.
*
* @param sortOrder
* @param sortOrder Iceberg sort order
* @return Gravitino sort order
*/
public static SortOrder[] fromSortOrder(org.apache.iceberg.SortOrder sortOrder) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -347,8 +347,8 @@ public static Namespace getIcebergNamespace(String... level) {
* Gravitino only supports tables managed with a single level hierarchy, such as
* `{namespace}.{table}`, so we need to perform truncation here.
*
* @param namespace
* @param name
* @param namespace The Gravitino name space
* @param name The table name
* @return Iceberg TableIdentifier
*/
public static TableIdentifier buildIcebergTableIdentifier(
Expand Down
Loading