Indhumathi27 commented on a change in pull request #4239:
URL: https://github.com/apache/carbondata/pull/4239#discussion_r768320746
##########
File path: docs/addsegment-guide.md
##########
@@ -27,16 +27,144 @@ Heterogeneous format segments aims to solve this problem
by avoiding data conver
### Add segment with path and format
Users can add the existing data as a segment to the carbon table provided the
schema of the data
and the carbon table should be the same.
+
+ Syntax
+
+ ```
+ ALTER TABLE [db_name.]table_name ADD SEGMENT
OPTIONS(property_name=property_value, ...)
+ ```
+
+**Supported properties:**
+
+| Property | Description
|
+| ------------------------------------------------------------ |
------------------------------------------------------------ |
+| [PATH](#path) | User external old table path |
+| [FORMAT](#format) | User external old table file format |
+| [PARTITION](#partition) | Partition info for partition table , should be
form of "a:int, b:string" |
+
+
+-
+ You can use the following options to add segment:
+
+ - ##### PATH:
+ User old table path.
+
+ ```
+ OPTIONS('PATH'='hdfs://usr/oldtable')
+ ```
+
+ - ##### FORMAT:
+ User old table file format. eg : parquet, orc
Review comment:
please format this line.
```suggestion
User old table file format. eg : parquet, orc
```
##########
File path: docs/addsegment-guide.md
##########
@@ -27,16 +27,144 @@ Heterogeneous format segments aims to solve this problem
by avoiding data conver
### Add segment with path and format
Users can add the existing data as a segment to the carbon table provided the
schema of the data
and the carbon table should be the same.
+
+ Syntax
+
+ ```
+ ALTER TABLE [db_name.]table_name ADD SEGMENT
OPTIONS(property_name=property_value, ...)
+ ```
+
+**Supported properties:**
+
+| Property | Description
|
+| ------------------------------------------------------------ |
------------------------------------------------------------ |
+| [PATH](#path) | User external old table path |
+| [FORMAT](#format) | User external old table file format |
+| [PARTITION](#partition) | Partition info for partition table , should be
form of "a:int, b:string" |
+
+
+-
+ You can use the following options to add segment:
+
+ - ##### PATH:
+ User old table path.
+
+ ```
+ OPTIONS('PATH'='hdfs://usr/oldtable')
+ ```
+
+ - ##### FORMAT:
+ User old table file format. eg : parquet, orc
+
+ ```
+ OPTIONS('FORMAT'='parquet')
+ ```
+ - ##### PARTITION:
+ Partition info for partition table , should be form of "a:int, b:string"
Review comment:
```suggestion
Partition info for partition table , should be form of "a:int, b:string"
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@carbondata.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
