Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2824#discussion_r227191521
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1371,16 +1371,15 @@
public static final String CARBON_SECURE_DICTIONARY_SERVER_DEFAULT =
"true";
/**
- * whether to use multi directories when loading data,
- * the main purpose is to avoid single-disk-hot-spot
+ * whether to use yarn's local dir the main purpose is to avoid single
disk hot spot
*/
@CarbonProperty
- public static final String CARBON_USE_MULTI_TEMP_DIR =
"carbon.use.multiple.temp.dir";
+ public static final String CARBON_USE_YARN_LOCAL_DIR =
"carbon.use.local.dir";
--- End diff --
ok. Then I think it is better change the variable name only, we can use the
existing variable value
---
