You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@polaris.apache.org by yu...@apache.org on 2026/08/31 05:24:05 UTC

(polaris) branch main updated: Clarify default table-like location documentation (#5414)

This is an automated email from the ASF dual-hosted git repository.

flyrain pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new d30a795960 Clarify default table-like location documentation (#5414)
d30a795960 is described below

commit d30a795960e4b63c8bf8af40adcf96abe70f53c3
Author: Yufei Gu <yu...@apache.org>
AuthorDate: Sun Aug 30 22:24:00 2026 -0700

    Clarify default table-like location documentation (#5414)
---
 .../apache/polaris/service/catalog/iceberg/LocalIcebergCatalog.java  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/LocalIcebergCatalog.java b/runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/LocalIcebergCatalog.java
index 1fcdadfefc..58044461c0 100644
--- a/runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/LocalIcebergCatalog.java
+++ b/runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/LocalIcebergCatalog.java
@@ -510,6 +510,11 @@ public class LocalIcebergCatalog extends BaseMetastoreViewCatalog
     return newTableOps(tableIdentifier, makeMetadataCurrentOnCommit);
   }
 
+  /**
+   * Please note: This method does NOT return the catalog's default warehouse location. It returns
+   * the table-like entity's location, derived from its namespace or the catalog warehouse. The
+   * method name is inherited from Iceberg and cannot be changed by this implementation.
+   */
   @Override
   protected String defaultWarehouseLocation(TableIdentifier tableIdentifier) {
     String prefixedLocation = applyDefaultLocationObjectStoragePrefix(tableIdentifier, null);