You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by QiangCai <gi...@git.apache.org> on 2018/03/01 08:20:17 UTC
[GitHub] carbondata pull request #1991: [CARBONDATA-2196]carbontable should be taken ...
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1991#discussion_r171487297
--- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonDataProcessorUtil.java ---
@@ -117,22 +117,25 @@ public static void createLocations(String[] locations) {
}
}
}
+
/**
+ *
* This method will form the local data folder store location
*
- * @param databaseName
- * @param tableName
+ * @param carbonTable
* @param taskId
* @param partitionId
* @param segmentId
+ * @param isCompactionFlow
+ * @param isAltPartitionFlow
* @return
*/
- public static String[] getLocalDataFolderLocation(String databaseName, String tableName,
+ public static String[] getLocalDataFolderLocation(CarbonTable carbonTable, String tableName,
--- End diff --
not require tableName
---