maheshrajus commented on a change in pull request #4164:
URL: https://github.com/apache/carbondata/pull/4164#discussion_r665089655



##########
File path: docs/index/secondary-index-guide.md
##########
@@ -118,9 +118,11 @@ also load data to the SI table with the same number of 
segments as the main tabl
 the main table will also load data to the SI table.
 
  **NOTE**:
- * In case of data load failure to the SI table, then we make the SI table 
disable by setting a hive serde
- property. The subsequent main table load will load the old failed loads along 
with current load and
- makes the SI table enable and available for query.
+ * In case of data load failure to the SI table, segments of main table and SI 
table will not be in sync.

Review comment:
       can we give one example for this scenario for easy understanding ? 
   ex:
   segment 1 loaded successfully to maintable and SI
   segment 2 loaded successfully to maintable and SI
   segment 3 failed to load. 
   show segment differences between maintable and SI table.
   so in this case for segment 1 and segment 2 pruned by SI and segment 3 
filter with help of maintable.




-- 
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: [email protected]

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


Reply via email to