[GitHub] carbondata pull request #3030: [HOTFIX] Optimize the code style in csdk/sdk ...

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495293
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    --- End diff --
   
    create to Create


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495349
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    --- End diff --
   
    remove one blank before 'can'


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495374
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    --- End diff --
   
    create to Create


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495396
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + *
    + */
    +void builder(JNIEnv *env, char *path);
     ```
     
     ```
    -    /**
    -     * Configure the projection column names of carbon reader
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject projection(int argc, char *argv[]);
    +/**
    + * Configure the projection column names of carbon reader
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + */
    +jobject projection(int argc, char *argv[]);
     ```
     
     ```
    -    /**
    -     *  build carbon reader with argument vector
    -     *  it support multiple parameter
    -     *  like: key=value
    -     *  for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     **/
    -    jobject withHadoopConf(int argc, char *argv[]);
    +/**
    + * build carbon reader with argument vector
    --- End diff --
   
    build to Build


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495429
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + *
    + */
    +void builder(JNIEnv *env, char *path);
     ```
     
     ```
    -    /**
    -     * Configure the projection column names of carbon reader
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject projection(int argc, char *argv[]);
    +/**
    + * Configure the projection column names of carbon reader
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + */
    +jobject projection(int argc, char *argv[]);
     ```
     
     ```
    -    /**
    -     *  build carbon reader with argument vector
    -     *  it support multiple parameter
    -     *  like: key=value
    -     *  for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     **/
    -    jobject withHadoopConf(int argc, char *argv[]);
    +/**
    + * build carbon reader with argument vector
    + * it support multiple parameter
    --- End diff --
   
    support to supports


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495542
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + *
    + */
    +void builder(JNIEnv *env, char *path);
     ```
     
     ```
    -    /**
    -     * Configure the projection column names of carbon reader
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject projection(int argc, char *argv[]);
    +/**
    + * Configure the projection column names of carbon reader
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + */
    +jobject projection(int argc, char *argv[]);
     ```
     
     ```
    -    /**
    -     *  build carbon reader with argument vector
    -     *  it support multiple parameter
    -     *  like: key=value
    -     *  for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     **/
    -    jobject withHadoopConf(int argc, char *argv[]);
    +/**
    + * build carbon reader with argument vector
    + * it support multiple parameter
    + * like: key=value
    + * for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + *
    + */
    +jobject withHadoopConf(int argc, char *argv[]);
     ```
     
     ```
    -   /**
    -     * Sets the batch size of records to read
    -     *
    -     * @param batch batch size
    -     * @return CarbonReaderBuilder object
    -     */
    -    void withBatch(int batch);
    +/**
    + * Sets the batch size of records to read
    + *
    + * @param batch batch size
    + * @return CarbonReaderBuilder object
    + */
    +void withBatch(int batch);
     ```
     
     ```
    -    /**
    -     * Configure Row Record Reader for reading.
    -     */
    -    void withRowRecordReader();
    +/**
    + * Configure Row Record Reader for reading.
    + */
    +void withRowRecordReader();
     ```
     
     ```
    -    /**
    -     * build carbonReader object for reading data
    -     * it support read data from load disk
    -     *
    -     * @return carbonReader object
    -     */
    -    jobject build();
    +/**
    + * build carbonReader object for reading data
    + * it support read data from load disk
    --- End diff --
   
    build to Build
    recommend: it supports to read data from load disk


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495567
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + *
    + */
    +void builder(JNIEnv *env, char *path);
     ```
     
     ```
    -    /**
    -     * Configure the projection column names of carbon reader
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject projection(int argc, char *argv[]);
    +/**
    + * Configure the projection column names of carbon reader
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + */
    +jobject projection(int argc, char *argv[]);
     ```
     
     ```
    -    /**
    -     *  build carbon reader with argument vector
    -     *  it support multiple parameter
    -     *  like: key=value
    -     *  for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     **/
    -    jobject withHadoopConf(int argc, char *argv[]);
    +/**
    + * build carbon reader with argument vector
    + * it support multiple parameter
    + * like: key=value
    + * for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + *
    + */
    +jobject withHadoopConf(int argc, char *argv[]);
     ```
     
     ```
    -   /**
    -     * Sets the batch size of records to read
    -     *
    -     * @param batch batch size
    -     * @return CarbonReaderBuilder object
    -     */
    -    void withBatch(int batch);
    +/**
    + * Sets the batch size of records to read
    + *
    + * @param batch batch size
    + * @return CarbonReaderBuilder object
    + */
    +void withBatch(int batch);
     ```
     
     ```
    -    /**
    -     * Configure Row Record Reader for reading.
    -     */
    -    void withRowRecordReader();
    +/**
    + * Configure Row Record Reader for reading.
    + */
    +void withRowRecordReader();
     ```
     
     ```
    -    /**
    -     * build carbonReader object for reading data
    -     * it support read data from load disk
    -     *
    -     * @return carbonReader object
    -     */
    -    jobject build();
    +/**
    + * build carbonReader object for reading data
    + * it support read data from load disk
    + *
    + * @return carbonReader object
    + */
    +jobject build();
     ```
     
     ```
    -    /**
    -     * Whether it has next row data
    -     *
    -     * @return boolean value, if it has next row, return true. if it hasn't next row, return false.
    -     */
    -    jboolean hasNext();
    +/**
    + * Whether it has next row data
    + *
    + * @return boolean value, if it has next row, return true. if it hasn't next row, return false.
    + */
    +jboolean hasNext();
     ```
     
     ```
    -    /**
    -     * read next carbonRow from data
    -     * @return carbonRow object of one row
    -     */
    -     jobject readNextRow();
    +/**
    + * read next carbonRow from data
    --- End diff --
   
    read to Read


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495589
 
    --- Diff: docs/csdk-guide.md ---
    @@ -43,114 +43,116 @@ C++ SDK support read batch row. User can set batch by using withBatch(int batch)
     ## API List
     ### CarbonReader
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * @param tableName table name
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject builder(JNIEnv *env, char *path, char *tableName);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + * @param tableName table name
    + * @return CarbonReaderBuilder object
    + */
    +jobject builder(JNIEnv *env, char *path, char *tableName);
     ```
     
     ```
    -    /**
    -     * create a CarbonReaderBuilder object for building carbonReader,
    -     * CarbonReaderBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @param path data store path
    -     * */
    -    void builder(JNIEnv *env, char *path);
    +/**
    + * create a CarbonReaderBuilder object for building carbonReader,
    + * CarbonReaderBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @param path data store path
    + *
    + */
    +void builder(JNIEnv *env, char *path);
     ```
     
     ```
    -    /**
    -     * Configure the projection column names of carbon reader
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     */
    -    jobject projection(int argc, char *argv[]);
    +/**
    + * Configure the projection column names of carbon reader
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + */
    +jobject projection(int argc, char *argv[]);
     ```
     
     ```
    -    /**
    -     *  build carbon reader with argument vector
    -     *  it support multiple parameter
    -     *  like: key=value
    -     *  for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    -     *
    -     * @param argc argument counter
    -     * @param argv argument vector
    -     * @return CarbonReaderBuilder object
    -     **/
    -    jobject withHadoopConf(int argc, char *argv[]);
    +/**
    + * build carbon reader with argument vector
    + * it support multiple parameter
    + * like: key=value
    + * for example: fs.s3a.access.key=XXXX, XXXX is user's access key value
    + *
    + * @param argc argument counter
    + * @param argv argument vector
    + * @return CarbonReaderBuilder object
    + *
    + */
    +jobject withHadoopConf(int argc, char *argv[]);
     ```
     
     ```
    -   /**
    -     * Sets the batch size of records to read
    -     *
    -     * @param batch batch size
    -     * @return CarbonReaderBuilder object
    -     */
    -    void withBatch(int batch);
    +/**
    + * Sets the batch size of records to read
    + *
    + * @param batch batch size
    + * @return CarbonReaderBuilder object
    + */
    +void withBatch(int batch);
     ```
     
     ```
    -    /**
    -     * Configure Row Record Reader for reading.
    -     */
    -    void withRowRecordReader();
    +/**
    + * Configure Row Record Reader for reading.
    + */
    +void withRowRecordReader();
     ```
     
     ```
    -    /**
    -     * build carbonReader object for reading data
    -     * it support read data from load disk
    -     *
    -     * @return carbonReader object
    -     */
    -    jobject build();
    +/**
    + * build carbonReader object for reading data
    + * it support read data from load disk
    + *
    + * @return carbonReader object
    + */
    +jobject build();
     ```
     
     ```
    -    /**
    -     * Whether it has next row data
    -     *
    -     * @return boolean value, if it has next row, return true. if it hasn't next row, return false.
    -     */
    -    jboolean hasNext();
    +/**
    + * Whether it has next row data
    + *
    + * @return boolean value, if it has next row, return true. if it hasn't next row, return false.
    + */
    +jboolean hasNext();
     ```
     
     ```
    -    /**
    -     * read next carbonRow from data
    -     * @return carbonRow object of one row
    -     */
    -     jobject readNextRow();
    +/**
    + * read next carbonRow from data
    + * @return carbonRow object of one row
    + */
    +jobject readNextRow();
     ```
     
     ```
    -    /**
    -     * read Next Batch Row
    -     *
    -     * @return rows
    -     */
    -    jobjectArray readNextBatchRow();
    +/**
    + * read Next Batch Row
    + *
    + * @return rows
    + */
    +jobjectArray readNextBatchRow();
     ```
     
     ```
    -    /**
    -     * close the carbon reader
    -     *
    -     * @return  boolean value
    -     */
    -    jboolean close();
    +/**
    + * close the carbon reader
    --- End diff --
   
    close to Close


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495612
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    --- End diff --
   
    create to Create


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495633
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    + * CarbonWriterBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @return CarbonWriterBuilder object
    + */
    +void builder(JNIEnv *env);
    +```
    +
    +```
    +/**
    + * Sets the output path of the writer builder
    + *
    + * @param path is the absolute path where output files are written
    + * This method must be called when building CarbonWriterBuilder
    + * @return updated CarbonWriterBuilder
    + */
    +void outputPath(char *path);
    +```
    +
    +```
    +/**
    + * sets the list of columns that needs to be in sorted order
    --- End diff --
   
    sets to Set


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495649
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    + * CarbonWriterBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @return CarbonWriterBuilder object
    + */
    +void builder(JNIEnv *env);
    +```
    +
    +```
    +/**
    + * Sets the output path of the writer builder
    + *
    + * @param path is the absolute path where output files are written
    + * This method must be called when building CarbonWriterBuilder
    + * @return updated CarbonWriterBuilder
    + */
    +void outputPath(char *path);
    +```
    +
    +```
    +/**
    + * sets the list of columns that needs to be in sorted order
    + *
    + * @param argc argc argument counter, the number of projection column
    + * @param argv argv is a string array of columns that needs to be sorted.
    + *                  If it is null or by default all dimensions are selected for sorting
    + *                  If it is empty array, no columns are sorted
    + */
    +void sortBy(int argc, char *argv[]);
    +```
    +
    +```
    +/**
    + * configure the schema with json style schema
    --- End diff --
   
    configure to Configure


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495697
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    + * CarbonWriterBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @return CarbonWriterBuilder object
    + */
    +void builder(JNIEnv *env);
    +```
    +
    +```
    +/**
    + * Sets the output path of the writer builder
    + *
    + * @param path is the absolute path where output files are written
    + * This method must be called when building CarbonWriterBuilder
    + * @return updated CarbonWriterBuilder
    + */
    +void outputPath(char *path);
    +```
    +
    +```
    +/**
    + * sets the list of columns that needs to be in sorted order
    + *
    + * @param argc argc argument counter, the number of projection column
    + * @param argv argv is a string array of columns that needs to be sorted.
    + *                  If it is null or by default all dimensions are selected for sorting
    + *                  If it is empty array, no columns are sorted
    + */
    +void sortBy(int argc, char *argv[]);
    +```
    +
    +```
    +/**
    + * configure the schema with json style schema
    + *
    + * @param jsonSchema json style schema
    + * @return updated CarbonWriterBuilder
    + */
    +void withCsvInput(char *jsonSchema);
    +```
    +
    +```
    +/**
    + * Updates the hadoop configuration with the given key value
    + *
    + * @param key key word
    + * @param value value
    + * @return CarbonWriterBuilder object
    + */
    +void withHadoopConf(char *key, char *value);
    +```
    +
    +```
    +/**
    + *  To support the table properties for writer
    + *
    + * @param key properties key
    + * @param value properties value
    + */
    +void withTableProperty(char *key, char *value);
    +```
    +
    +```
    +/**
    + * To support the load options for C++ sdk writer
    + *
    + * @param options key,value pair of load options.
    + * supported keys values are
    + * a. bad_records_logger_enable -- true (write into separate logs), false
    + * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    + * c. bad_record_path -- path
    + * d. dateformat -- same as JAVA SimpleDateFormat
    + * e. timestampformat -- same as JAVA SimpleDateFormat
    + * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    + * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    + * h. quotechar
    + * i. escapechar
    + *
    + * Default values are as follows.
    + *
    + * a. bad_records_logger_enable -- "false"
    + * b. bad_records_action -- "FAIL"
    + * c. bad_record_path -- ""
    + * d. dateformat -- "" , uses from carbon.properties file
    + * e. timestampformat -- "", uses from carbon.properties file
    + * f. complex_delimiter_level_1 -- "$"
    + * g. complex_delimiter_level_2 -- ":"
    + * h. quotechar -- "\""
    + * i. escapechar -- "\\"
    + *
    + * @return updated CarbonWriterBuilder
    + */
    +void withLoadOption(char *key, char *value);
     ```
     
     ```
    -    /**
    -     * sets the taskNo for the writer. CSDKs concurrently running
    -     * will set taskNo in order to avoid conflicts in file's name during write.
    -     *
    -     * @param taskNo is the TaskNo user wants to specify.
    -     *               by default it is system time in nano seconds.
    -     */
    -    void taskNo(long taskNo);
    +/**
    + * sets the taskNo for the writer. CSDKs concurrently running
    --- End diff --
   
    sets to Set


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495752
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    + * CarbonWriterBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @return CarbonWriterBuilder object
    + */
    +void builder(JNIEnv *env);
    +```
    +
    +```
    +/**
    + * Sets the output path of the writer builder
    + *
    + * @param path is the absolute path where output files are written
    + * This method must be called when building CarbonWriterBuilder
    + * @return updated CarbonWriterBuilder
    + */
    +void outputPath(char *path);
    +```
    +
    +```
    +/**
    + * sets the list of columns that needs to be in sorted order
    + *
    + * @param argc argc argument counter, the number of projection column
    + * @param argv argv is a string array of columns that needs to be sorted.
    + *                  If it is null or by default all dimensions are selected for sorting
    + *                  If it is empty array, no columns are sorted
    + */
    +void sortBy(int argc, char *argv[]);
    +```
    +
    +```
    +/**
    + * configure the schema with json style schema
    + *
    + * @param jsonSchema json style schema
    + * @return updated CarbonWriterBuilder
    + */
    +void withCsvInput(char *jsonSchema);
    +```
    +
    +```
    +/**
    + * Updates the hadoop configuration with the given key value
    + *
    + * @param key key word
    + * @param value value
    + * @return CarbonWriterBuilder object
    + */
    +void withHadoopConf(char *key, char *value);
    +```
    +
    +```
    +/**
    + *  To support the table properties for writer
    + *
    + * @param key properties key
    + * @param value properties value
    + */
    +void withTableProperty(char *key, char *value);
    +```
    +
    +```
    +/**
    + * To support the load options for C++ sdk writer
    + *
    + * @param options key,value pair of load options.
    + * supported keys values are
    + * a. bad_records_logger_enable -- true (write into separate logs), false
    + * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    + * c. bad_record_path -- path
    + * d. dateformat -- same as JAVA SimpleDateFormat
    + * e. timestampformat -- same as JAVA SimpleDateFormat
    + * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    + * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    + * h. quotechar
    + * i. escapechar
    + *
    + * Default values are as follows.
    + *
    + * a. bad_records_logger_enable -- "false"
    + * b. bad_records_action -- "FAIL"
    + * c. bad_record_path -- ""
    + * d. dateformat -- "" , uses from carbon.properties file
    + * e. timestampformat -- "", uses from carbon.properties file
    + * f. complex_delimiter_level_1 -- "$"
    + * g. complex_delimiter_level_2 -- ":"
    + * h. quotechar -- "\""
    + * i. escapechar -- "\\"
    + *
    + * @return updated CarbonWriterBuilder
    + */
    +void withLoadOption(char *key, char *value);
     ```
     
     ```
    -    /**
    -     * sets the taskNo for the writer. CSDKs concurrently running
    -     * will set taskNo in order to avoid conflicts in file's name during write.
    -     *
    -     * @param taskNo is the TaskNo user wants to specify.
    -     *               by default it is system time in nano seconds.
    -     */
    -    void taskNo(long taskNo);
    +/**
    + * sets the taskNo for the writer. CSDKs concurrently running
    + * will set taskNo in order to avoid conflicts in file's name during write.
    + *
    + * @param taskNo is the TaskNo user wants to specify.
    + *               by default it is system time in nano seconds.
    + */
    +void taskNo(long taskNo);
     ```
     
     ```
    -    /**
    -     * to set the timestamp in the carbondata and carbonindex index files
    -     *
    -     * @param timestamp is a timestamp to be used in the carbondata and carbonindex index files.
    -     * By default set to zero.
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void uniqueIdentifier(long timestamp);
    +/**
    + * to set the timestamp in the carbondata and carbonindex index files
    --- End diff --
   
    'to set' to Set


---

[qiuchenjian-2]

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

    https://github.com/apache/carbondata/pull/3030#discussion_r244495764
 
    --- Diff: docs/csdk-guide.md ---
    @@ -172,361 +174,360 @@ release the memory and destroy JVM.
     ## API List
     ### CarbonWriter
     ```
    -    /**
    -     * create a CarbonWriterBuilder object for building carbonWriter,
    -     * CarbonWriterBuilder object  can configure different parameter
    -     *
    -     * @param env JNIEnv
    -     * @return CarbonWriterBuilder object
    -     */
    -    void builder(JNIEnv *env);
    -```
    -
    -```
    -    /**
    -     * Sets the output path of the writer builder
    -     *
    -     * @param path is the absolute path where output files are written
    -     * This method must be called when building CarbonWriterBuilder
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void outputPath(char *path);
    -```
    -
    -```
    -    /**
    -      * sets the list of columns that needs to be in sorted order
    -      *
    -      * @param argc argc argument counter, the number of projection column
    -      * @param argv argv is a string array of columns that needs to be sorted.
    -      *                  If it is null or by default all dimensions are selected for sorting
    -      *                  If it is empty array, no columns are sorted
    -      */
    -    void sortBy(int argc, char *argv[]);
    -```
    -
    -```
    -    /**
    -     * configure the schema with json style schema
    -     *
    -     * @param jsonSchema json style schema
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withCsvInput(char *jsonSchema);
    -```
    -
    -```
    -    /**
    -    * Updates the hadoop configuration with the given key value
    -    *
    -    * @param key key word
    -    * @param value value
    -    * @return CarbonWriterBuilder object
    -    */
    -    void withHadoopConf(char *key, char *value);
    -```
    -
    -```
    - /**
    -     *  To support the table properties for writer
    -     *
    -     * @param key properties key
    -     * @param value properties value
    -     */
    -    void withTableProperty(char *key, char *value);
    -```
    -
    -```
    -    /**
    -     * To support the load options for C++ sdk writer
    -     *
    -     * @param options key,value pair of load options.
    -     * supported keys values are
    -     * a. bad_records_logger_enable -- true (write into separate logs), false
    -     * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    -     * c. bad_record_path -- path
    -     * d. dateformat -- same as JAVA SimpleDateFormat
    -     * e. timestampformat -- same as JAVA SimpleDateFormat
    -     * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    -     * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    -     * h. quotechar
    -     * i. escapechar
    -     *
    -     * Default values are as follows.
    -     *
    -     * a. bad_records_logger_enable -- "false"
    -     * b. bad_records_action -- "FAIL"
    -     * c. bad_record_path -- ""
    -     * d. dateformat -- "" , uses from carbon.properties file
    -     * e. timestampformat -- "", uses from carbon.properties file
    -     * f. complex_delimiter_level_1 -- "$"
    -     * g. complex_delimiter_level_2 -- ":"
    -     * h. quotechar -- "\""
    -     * i. escapechar -- "\\"
    -     *
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withLoadOption(char *key, char *value);
    +/**
    + * create a CarbonWriterBuilder object for building carbonWriter,
    + * CarbonWriterBuilder object  can configure different parameter
    + *
    + * @param env JNIEnv
    + * @return CarbonWriterBuilder object
    + */
    +void builder(JNIEnv *env);
    +```
    +
    +```
    +/**
    + * Sets the output path of the writer builder
    + *
    + * @param path is the absolute path where output files are written
    + * This method must be called when building CarbonWriterBuilder
    + * @return updated CarbonWriterBuilder
    + */
    +void outputPath(char *path);
    +```
    +
    +```
    +/**
    + * sets the list of columns that needs to be in sorted order
    + *
    + * @param argc argc argument counter, the number of projection column
    + * @param argv argv is a string array of columns that needs to be sorted.
    + *                  If it is null or by default all dimensions are selected for sorting
    + *                  If it is empty array, no columns are sorted
    + */
    +void sortBy(int argc, char *argv[]);
    +```
    +
    +```
    +/**
    + * configure the schema with json style schema
    + *
    + * @param jsonSchema json style schema
    + * @return updated CarbonWriterBuilder
    + */
    +void withCsvInput(char *jsonSchema);
    +```
    +
    +```
    +/**
    + * Updates the hadoop configuration with the given key value
    + *
    + * @param key key word
    + * @param value value
    + * @return CarbonWriterBuilder object
    + */
    +void withHadoopConf(char *key, char *value);
    +```
    +
    +```
    +/**
    + *  To support the table properties for writer
    + *
    + * @param key properties key
    + * @param value properties value
    + */
    +void withTableProperty(char *key, char *value);
    +```
    +
    +```
    +/**
    + * To support the load options for C++ sdk writer
    + *
    + * @param options key,value pair of load options.
    + * supported keys values are
    + * a. bad_records_logger_enable -- true (write into separate logs), false
    + * b. bad_records_action -- FAIL, FORCE, IGNORE, REDIRECT
    + * c. bad_record_path -- path
    + * d. dateformat -- same as JAVA SimpleDateFormat
    + * e. timestampformat -- same as JAVA SimpleDateFormat
    + * f. complex_delimiter_level_1 -- value to Split the complexTypeData
    + * g. complex_delimiter_level_2 -- value to Split the nested complexTypeData
    + * h. quotechar
    + * i. escapechar
    + *
    + * Default values are as follows.
    + *
    + * a. bad_records_logger_enable -- "false"
    + * b. bad_records_action -- "FAIL"
    + * c. bad_record_path -- ""
    + * d. dateformat -- "" , uses from carbon.properties file
    + * e. timestampformat -- "", uses from carbon.properties file
    + * f. complex_delimiter_level_1 -- "$"
    + * g. complex_delimiter_level_2 -- ":"
    + * h. quotechar -- "\""
    + * i. escapechar -- "\\"
    + *
    + * @return updated CarbonWriterBuilder
    + */
    +void withLoadOption(char *key, char *value);
     ```
     
     ```
    -    /**
    -     * sets the taskNo for the writer. CSDKs concurrently running
    -     * will set taskNo in order to avoid conflicts in file's name during write.
    -     *
    -     * @param taskNo is the TaskNo user wants to specify.
    -     *               by default it is system time in nano seconds.
    -     */
    -    void taskNo(long taskNo);
    +/**
    + * sets the taskNo for the writer. CSDKs concurrently running
    + * will set taskNo in order to avoid conflicts in file's name during write.
    + *
    + * @param taskNo is the TaskNo user wants to specify.
    + *               by default it is system time in nano seconds.
    + */
    +void taskNo(long taskNo);
     ```
     
     ```
    -    /**
    -     * to set the timestamp in the carbondata and carbonindex index files
    -     *
    -     * @param timestamp is a timestamp to be used in the carbondata and carbonindex index files.
    -     * By default set to zero.
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void uniqueIdentifier(long timestamp);
    +/**
    + * to set the timestamp in the carbondata and carbonindex index files
    + *
    + * @param timestamp is a timestamp to be used in the carbondata and carbonindex index files.
    + * By default set to zero.
    + * @return updated CarbonWriterBuilder
    + */
    +void uniqueIdentifier(long timestamp);
     ```
     
     ```
    -    /**
    -     * To make c++ sdk writer thread safe.
    -     *
    -     * @param numOfThreads should number of threads in which writer is called in multi-thread scenario
    -     *                      default C++ sdk writer is not thread safe.
    -     *                      can use one writer instance in one thread only.
    -     */
    -    void withThreadSafe(short numOfThreads) ;
    +/**
    + * To make c++ sdk writer thread safe.
    + *
    + * @param numOfThreads should number of threads in which writer is called in multi-thread scenario
    + *                      default C++ sdk writer is not thread safe.
    + *                      can use one writer instance in one thread only.
    + */
    +void withThreadSafe(short numOfThreads) ;
     ```
     
     ```
    -    /**
    -     * To set the carbondata file size in MB between 1MB-2048MB
    -     *
    -     * @param blockSize is size in MB between 1MB to 2048 MB
    -     * default value is 1024 MB
    -     */
    -    void withBlockSize(int blockSize);
    +/**
    + * To set the carbondata file size in MB between 1MB-2048MB
    + *
    + * @param blockSize is size in MB between 1MB to 2048 MB
    + * default value is 1024 MB
    + */
    +void withBlockSize(int blockSize);
     ```
     
     ```
    -    /**
    -     * To set the blocklet size of CarbonData file
    -     *
    -     * @param blockletSize is blocklet size in MB
    -     *        default value is 64 MB
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void withBlockletSize(int blockletSize);
    +/**
    + * To set the blocklet size of CarbonData file
    + *
    + * @param blockletSize is blocklet size in MB
    + *        default value is 64 MB
    + * @return updated CarbonWriterBuilder
    + */
    +void withBlockletSize(int blockletSize);
     ```
     
     ```
    -    /**
    -     * @param localDictionaryThreshold is localDictionaryThreshold, default is 10000
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void localDictionaryThreshold(int localDictionaryThreshold);
    +/**
    + * @param localDictionaryThreshold is localDictionaryThreshold, default is 10000
    + * @return updated CarbonWriterBuilder
    + */
    +void localDictionaryThreshold(int localDictionaryThreshold);
     ```
     
     ```
    -    /**
    -     * @param enableLocalDictionary enable local dictionary, default is false
    -     * @return updated CarbonWriterBuilder
    -     */
    -    void enableLocalDictionary(bool enableLocalDictionary);
    +/**
    + * @param enableLocalDictionary enable local dictionary, default is false
    + * @return updated CarbonWriterBuilder
    + */
    +void enableLocalDictionary(bool enableLocalDictionary);
     ```
     
     ```
    -    /**
    -     * @param appName appName which is writing the carbondata files
    -     */
    -    void writtenBy(char *appName);
    +/**
    + * @param appName appName which is writing the carbondata files
    + */
    +void writtenBy(char *appName);
     ```
     
     ```
    -    /**
    -     * build carbonWriter object for writing data
    -     * it support write data from load disk
    -     *
    -     * @return carbonWriter object
    -     */
    -    void build();
    +/**
    + * build carbonWriter object for writing data
    --- End diff --
   
    build to Build


---

[qiuchenjian-2]

Github user zzcclp commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    There are some other capitalization problems, please check and fix.


---

[qiuchenjian-2]

Github user lamber-ken commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    > There are some other capitalization problems, please check and fix.
   
    thanks for detail review. I have update the doc


---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2083/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2288/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    Build Success with Spark 2.3.2, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/10337/



---

[qiuchenjian-2]

Github user xubo245 commented on the issue:

    https://github.com/apache/carbondata/pull/3030
 
    LGTM! Thanks for your contribution!


---