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;
    }


---

Reply via email to