[GitHub] [iceberg] wang-x-xia commented on a change in pull request #4221: Dell: Add Dell EMC EcsCatalog.

[GitBox <gi...@apache.org>]

wang-x-xia commented on a change in pull request #4221:
URL: https://github.com/apache/iceberg/pull/4221#discussion_r814494578



##########
File path: dell/src/main/java/org/apache/iceberg/dell/DellProperties.java
##########
@@ -38,6 +40,21 @@
    */
   public static final String ECS_S3_ENDPOINT = "ecs.s3.endpoint";
 
+  /**
+   * Catalog prefix is used to store catalog data. If not set, use {@link CatalogProperties#WAREHOUSE_LOCATION}.
+   * <p>
+   * The value is an EcsURI which like ecs://bucket/prefix.
+   */
+  public static final String ECS_CATALOG_PREFIX = "ecs.catalog.prefix";

Review comment:
       If the customer needs to separate the data and catalog metadata, that can set this config. 
   This config is not required. The default value is `CatalogProperties#WAREHOUSE_LOCATION`.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org