Skip to content

Commit a1d1176

Browse files
authored
Core: Clarify the purpose of REPLACE_NEW_LOCATION_PREFIX_WITH_CATALOG_DEFAULT_KEY (#2509)
1 parent d1a329f commit a1d1176

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

polaris-core/src/main/java/org/apache/polaris/core/entity/CatalogEntity.java

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,11 +61,18 @@ public class CatalogEntity extends PolarisEntity implements LocationBasedEntity
6161
// catalog, stored in the "properties" map.
6262
public static final String DEFAULT_BASE_LOCATION_KEY = "default-base-location";
6363

64-
// Specifies a prefix that will be replaced with the catalog's default-base-location whenever
65-
// it matches a specified new table or view location. For example, if the catalog base location
66-
// is "s3://my-bucket/base/location" and the prefix specified here is "file:/tmp" then any
67-
// new table attempting to specify a base location of "file:/tmp/ns1/ns2/table1" will be
68-
// translated into "s3://my-bucket/base/location/ns1/ns2/table1".
64+
/**
65+
* Test-only property that specifies a prefix that will be replaced with the catalog's
66+
* default-base-location whenever it matches a specified new table or view location.
67+
*
68+
* <p>For example, if the catalog base location is "s3://my-bucket/base/location" and the prefix
69+
* specified here is "file:/tmp" then any new table attempting to specify a base location of
70+
* "file:/tmp/ns1/ns2/table1" will be translated into
71+
* "s3://my-bucket/base/location/ns1/ns2/table1".
72+
*
73+
* <p><strong>WARNING:</strong> This property is intended for testing purposes only and should not
74+
* be used in production environments.
75+
*/
6976
public static final String REPLACE_NEW_LOCATION_PREFIX_WITH_CATALOG_DEFAULT_KEY =
7077
"replace-new-location-prefix-with-catalog-default";
7178

0 commit comments

Comments
 (0)