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

    https://github.com/apache/carbondata/pull/2336#discussion_r191328076
  
    --- Diff: 
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ---
    @@ -88,6 +89,14 @@ public static CarbonReaderBuilder builder(String 
tablePath, String tableName) {
         return new CarbonReaderBuilder(tablePath, tableName);
       }
     
    +  /**
    +   * Return a new {@link CarbonReaderBuilder} instance
    +   */
    +  public static CarbonReaderBuilder builder(String tablePath) {
    +    String uniqueName = "_temp" + UUID.randomUUID().toString();
    --- End diff --
    
    Suggest to use a more meaningful name, like `s"table@$tablePath"`


---

Reply via email to