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

    https://github.com/apache/carbondata/pull/2834#discussion_r227195246
  
    --- Diff: 
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java 
---
    @@ -412,10 +413,13 @@ private void setCsvHeader(CarbonLoadModel model) {
       public CarbonLoadModel buildLoadModel(Schema carbonSchema)
           throws IOException, InvalidLoadOptionException {
         timestamp = System.nanoTime();
    -    Set<String> longStringColumns = null;
    -    if (options != null && options.get("long_string_columns") != null) {
    -      longStringColumns =
    -          new 
HashSet<>(Arrays.asList(options.get("long_string_columns").toLowerCase().split(",")));
    +    Set<String> longStringColumns = new HashSet<>();
    +    if (options != null && 
options.get(CarbonCommonConstants.LONG_STRING_COLUMNS) != null) {
    --- End diff --
    
    OK, will fix it


---

Reply via email to