Skip to content

Commit fd6ee8f

Browse files
authored
[apache#7255] improve (trino-connector): Treat the version check for catalog names with dots as a warning. (apache#7256)
### What changes were proposed in this pull request? Treat the version check for catalog names with dots as a warning. ### Why are the changes needed? Fix: apache#7255 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Manually test
1 parent a4b0d7a commit fd6ee8f

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

  • trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog

trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/CatalogRegister.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -80,11 +80,11 @@ private void checkSupportCatalogNameWithMetalake(
8080
if (!config.simplifyCatalogNames()) {
8181
int version = Integer.parseInt(context.getSpiVersion());
8282
if (version < MIN_SUPPORT_CATALOG_NAME_WITH_METALAKE_TRINO_SPI_VERSION) {
83-
String errmsg =
84-
String.format(
85-
"Trino-%s does not support catalog name with dots, The minimal required version is Trino-%d",
86-
trinoVersion, MIN_SUPPORT_CATALOG_NAME_WITH_METALAKE_TRINO_SPI_VERSION);
87-
throw new TrinoException(GravitinoErrorCode.GRAVITINO_UNSUPPORTED_TRINO_VERSION, errmsg);
83+
LOG.warn(
84+
"Trino-{} does not support catalog name with dots, The minimal required version is Trino-{}."
85+
+ "Some errors may occur when using the USE <CATALOG>.<SCHEMA> statement in Trino",
86+
trinoVersion,
87+
MIN_SUPPORT_CATALOG_NAME_WITH_METALAKE_TRINO_SPI_VERSION);
8888
}
8989
}
9090
}

0 commit comments

Comments
 (0)