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

    https://github.com/apache/carbondata/pull/2054#discussion_r173842791
  
    --- Diff: 
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVCarbonWriterSuite.java
 ---
    @@ -68,13 +68,12 @@ public void testWriteFilesJsonSchema() throws 
IOException {
     
       private void writeFilesAndVerify(Schema schema, String path) {
         try {
    -      CarbonWriter writer = CarbonWriter.builder()
    -          .withSchema(schema)
    -          .outputPath(path)
    -          .buildWriterForCSVInput();
    +      CarbonWriter writer =
    +          
CarbonWriter.builder().withSchema(schema).outputPath(path).buildWriterForCSVInput();
     
           for (int i = 0; i < 100; i++) {
    -        writer.write(new String[]{"robot" + i, String.valueOf(i), 
String.valueOf((double) i / 2)});
    +        writer
    +            .write(new String[] { "robot" + i, String.valueOf(i), 
String.valueOf((double) i / 2) });
    --- End diff --
    
    do not modify it since no change


---

Reply via email to