Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2004#discussion_r170922780
--- 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 --
ok, it's fine
---