Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1914#discussion_r165674058
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -1891,7 +1892,8 @@ public static boolean validateValidIntType(String
value) {
* @return
*/
public static boolean isValidBadStorePath(String badRecordsLocation) {
- return !(null == badRecordsLocation || badRecordsLocation.length() ==
0);
+ if (StringUtils.isEmpty(badRecordsLocation)) return false;
+ else return isFileExists(checkAndAppendHDFSUrl(badRecordsLocation));
--- End diff --
Follow code formatting
if (StringUtils.isEmpty(badRecordsLocation)) {
return false;
}
---
