Login  Register

[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #4140: [CARBONDATA-4192] UT cases correction for validating the exception message correctly

Posted by GitBox on Jun 03, 2021; 5:42am
URL: http://apache-carbondata-dev-mailing-list-archive.168.s1.nabble.com/GitHub-carbondata-maheshrajus-opened-a-new-pull-request-4140-CARBONDATA-4192-UT-cases-correction-fory-tp108379p108545.html


Indhumathi27 commented on a change in pull request #4140:
URL: https://github.com/apache/carbondata/pull/4140#discussion_r644499287



##########
File path: docs/addsegment-guide.md
##########
@@ -17,14 +17,14 @@
 
 # Heterogeneous format segments in carbondata
 
-###Background
+### Background
 In the industry, many users already adopted to data with different formats like ORC, Parquet, JSON, CSV etc.,  
 If users want to migrate to Carbondata for better performance or for better features then there is no direct way.
 All the existing data needs to be converted to Carbondata to migrate.  
 This solution works out if the existing data is less, what if the existing data is more?  
 Heterogeneous format segments aims to solve this problem by avoiding data conversion.
 
-###Add segment with path and format
+### 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.
 

Review comment:
       Looks like syntax is wrong. Should be
   Alter table table_name add segment options (‘path’= 'hdfs://usr/oldtable','format'='parquet')




--
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.

For queries about this service, please contact Infrastructure at:
[hidden email]