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


---