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

    https://github.com/apache/carbondata/pull/2274#discussion_r189592810
  
    --- Diff: docs/sdk-writer-guide.md ---
    @@ -13,25 +13,33 @@ These SDK writer output contains just a carbondata and 
carbonindex files. No met
      
      import 
org.apache.carbondata.common.exceptions.sql.InvalidLoadOptionException;
      import org.apache.carbondata.core.metadata.datatype.DataTypes;
    + import org.apache.carbondata.core.util.CarbonProperties;
      import org.apache.carbondata.sdk.file.CarbonWriter;
      import org.apache.carbondata.sdk.file.CarbonWriterBuilder;
      import org.apache.carbondata.sdk.file.Field;
      import org.apache.carbondata.sdk.file.Schema;
      
      public class TestSdk {
    - 
    +
    +   // pass true or false while executing the main to use offheap memory or 
not
        public static void main(String[] args) throws IOException, 
InvalidLoadOptionException {
    -     testSdkWriter();
    +     if(args[0] != null) {
    --- End diff --
    
    yeah, this path also has problem. I raised a commit just 
now:https://github.com/xubo245/carbondata/commit/09f387717e5eabb01bf9f688c733e3342ce22799,
 it can run success, maybe you can refer


---

Reply via email to