Skip to content

[#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode #18418

[#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode

[#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode #18418

Triggered via pull request January 16, 2025 03:01
Status Cancelled
Total duration 1m 34s
Artifacts

build.yml

on: pull_request
changes
5s
changes
compile-check
0s
compile-check
spark-connector-build
0s
spark-connector-build
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 3 warnings
build (17): task ':catalogs:catalog-lakehouse-iceberg:spotlessJavaCheck'#L1
Execution failed for task ':catalogs:catalog-lakehouse-iceberg:spotlessJavaCheck': The following files had format violations: src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogOperations.java @@ -77,17 +77,14 @@ import·org.slf4j.Logger; import·org.slf4j.LoggerFactory; -/** -·*·Operations·for·interacting·with·an·Apache·Iceberg·catalog·in·Apache·Gravitino. -·*/ +/**·Operations·for·interacting·with·an·Apache·Iceberg·catalog·in·Apache·Gravitino.·*/ public·class·IcebergCatalogOperations·implements·CatalogOperations,·SupportsSchemas,·TableCatalog·{ ··private·static·final·String·ICEBERG_TABLE_DOES_NOT_EXIST_MSG·=·"Iceberg·table·does·not·exist:·%s"; ··public·static·final·Logger·LOG·=·LoggerFactory.getLogger(IcebergCatalogOperations.class); -··@VisibleForTesting -··IcebergCatalogWrapper·icebergCatalogWrapper; +··@VisibleForTesting·IcebergCatalogWrapper·icebergCatalogWrapper; ··private·IcebergCatalogWrapperHelper·icebergCatalogWrapperHelper; @@ -120,9 +117,7 @@ ········new·IcebergCatalogWrapperHelper(icebergCatalogWrapper.getCatalog()); ··} -··/** -···*·Closes·the·Iceberg·catalog·and·releases·the·associated·client·pool. -···*/ +··/**·Closes·the·Iceberg·catalog·and·releases·the·associated·client·pool.·*/ ··@OverRide ··public·void·close()·{ ····if·(null·!=·icebergCatalogWrapper)·{ @@ -161,11 +156,11 @@ ··/** ···*·Creates·a·new·schema·with·the·provided·identifier,·comment,·and·metadata. ···* -···*·@param·ident······The·identifier·of·the·schema·to·create. -···*·@param·comment····The·comment·for·the·schema. +···*·@param·ident·The·identifier·of·the·schema·to·create. +···*·@param·comment·The·comment·for·the·schema. ···*·@param·properties·The·properties·for·the·schema. ···*·@return·The·created·{@link·IcebergSchema}. -···*·@throws·NoSuchCatalogException·······If·the·provided·namespace·is·invalid·or·does·not·exist. +···*·@throws·NoSuchCatalogException·If·the·provided·namespace·is·invalid·or·does·not·exist. ···*·@throws·SchemaAlreadyExistsException·If·a·schema·with·the·same·name·already·exists. ···*/ ··@OverRide @@ -243,7 +238,7 @@ ··/** ···*·Alters·the·schema·with·the·provided·identifier·according·to·the·specified·changes. ... (104 more lines that didn't fit) Run './gradlew :catalogs:catalog-lakehouse-iceberg:spotlessApply' to fix these violations. at com.diffplug.gradle.spotless.SpotlessCheck.performAction(SpotlessCheck.java:111) at com.diffplug.gradle.spotless.SpotlessCheck.performAction(SpotlessCheck.java:53) at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:125) at org.gradle.api.internal.project.taskfactory.StandardTaskAction.doExecute(StandardTaskAction.java:58) at org.gradle.api.internal.project.taskfactory.StandardTaskAction.execute(StandardTaskAction.java:51) at org.gradle.api.internal.project.taskfactory.StandardTaskAction.execute(StandardTaskAction.java:29) at org.gradle.api.internal.tasks.execution.TaskExecution$3.run(TaskExecution.java:248) at org.gradle.internal.operations.DefaultBuildOperationRunner$1.execute(DefaultBuildOperationRunner.java:29) at org.gradle.internal.operations.DefaultBuildOperationRunner$1.execute(DefaultBuildOperationRunner.java:26) at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:66) at org.gradle.internal.operations.DefaultBuildOperationRunner$2.execute(DefaultBuildOperationRunner.java:59)
build (17)
Process completed with exit code 1.
build (8)
The job was canceled because "_17" failed.
build (8)
The operation was canceled.
build (11)
The job was canceled because "_17" failed.
build (11)
The operation was canceled.
changes
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build (17)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build (17)
No files were found with the provided path: build/reports catalogs/**/*.log catalogs/**/*.tar. No artifacts will be uploaded.