[GitHub] [hudi] bvaradar commented on a diff in pull request #8208: [HUDI-5903]: Introduce Glue sync configs

["bvaradar (via GitHub)" <gi...@apache.org>]

bvaradar commented on code in PR #8208:
URL: https://github.com/apache/hudi/pull/8208#discussion_r1145707752


##########
hudi-aws/src/main/java/org/apache/hudi/config/HoodieAWSConfig.java:
##########
@@ -63,6 +63,11 @@ public class HoodieAWSConfig extends HoodieConfig {
         .sinceVersion("0.10.0")
         .withDocumentation("AWS session token");
 
+  public static final ConfigProperty<String> AWS_REGION = ConfigProperty
+          .key("hoodie.aws.region")

Review Comment:
   Should this config be specific to Glue ? How are we planning to use this config for pointing to S3 in different region ? Will that be a separate config ? 



-- 
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: commits-unsubscribe@hudi.apache.org

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