Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2824#discussion_r226508384
--- 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 --
@jackylk This parameter is not new. If we changed the parameter name, for
upgrade scenario, the user has to change the configured parameters as well.
So I think we should keep the old parameter name here...
---
