nihal0107 edited a comment on pull request #4086:
URL: https://github.com/apache/carbondata/pull/4086#issuecomment-771464559


   > > If we enable the property `ENABLE_AUTO_LOAD_MERGE` then which segment id 
are we planning to show, the segment generated after compaction or before 
compaction? Better to add a test case for that scenario also.
   > 
   > > If we enable the property `ENABLE_AUTO_LOAD_MERGE` then which segment id 
are we planning to show, the segment generated after compaction or before 
compaction? Better to add a test case for that scenario also.
   > 
   > If we enable 'AUTO_LOAD_MERGE',then we return and show the segment id 
before compaction since the user would focus on his load operation. Test case 
has been added. Please review.
   
   We are showing the segment id because if we need to query on specific 
segment then this feature will be helpful. But if we will show the segment id 
before compaction and will query on specific segment then the operation will 
fail. Better to take the opinion in community.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to