Github user kumarvishal09 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2004#discussion_r170916613
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/path/CarbonTablePath.java ---
    @@ -75,7 +75,12 @@ public CarbonTablePath(CarbonTableIdentifier 
carbonTableIdentifier, String table
        * @param carbonFilePath
        */
       public static String getFolderContainingFile(String carbonFilePath) {
    -    return carbonFilePath.substring(0, carbonFilePath.lastIndexOf('/'));
    +    int lastIndex = carbonFilePath.lastIndexOf('/');
    +    // below code for handling windows environment
    +    if (-1 == lastIndex) {
    --- End diff --
    
     E:/xubo\idea  will never come either  it will be E:/xubo/idea or  
E:\xubo\idea ?


---

Reply via email to