rangareddy commented on issue #17269:
URL: https://github.com/apache/hudi/issues/17269#issuecomment-5365809566

   This issue was reviewed as part of the JIRA-migrated backlog triage 
(HUDI-8177).
   
   **Findings: still open.**
   
   The ask has two halves - validate the newer MDT index types (expression 
index, secondary index, partition stats), and generalise 
`HoodieMetadataTableValidator` so future index types are cheap to add.
   
   A useful signal on the second half: the per-index work is being done one 
index at a time and has not merged. Sibling #17275 (HUDI-8263) adds 
expression-index validation through PR #12151, still open. If the 
generalisation in this ticket had landed, adding one index type would not need 
its own PR - so (b) is clearly outstanding, and (a) is at best partial.
   
   Suggest doing the generalisation first and rebasing #12151 onto it, rather 
than landing per-index validators that then need reworking.
   
   Keeping this open.
   


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