Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1584#discussion_r159119279
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1420,6 +1425,16 @@
public static final String CARBON_UPDATE_SYNC_FOLDER_DEFAULT =
"/tmp/carbondata";
+ /**
+ * S3 Constants
+ */
+
+ public static final String S3_IMPLEMENTATION = "fs.s3a.impl";
--- End diff --
How about S3N support? I think this can be a property set by user, user can
choose S3 implementation class. The default class can be S3A
---
