nsivabalan commented on PR #5275: URL: https://github.com/apache/hudi/pull/5275#issuecomment-1095157951
In the interest of time for 0.11 release, here is my take. I haven't looked at the changes as such. but my 2 cents given the last min changes. We can ignore scheme evolution, CSI config changes (list of columns to index) for now. We can call out that CSI configs can be set only once and cannot be changed (list of cols to index), and may not work w/ schema evolution. enable and disable should be doable, just changing the list of columns to index on the fly is not feasible. just that we should not miss or regress any core flow by accomodating changes to support advanced use-cases like config changes and schema evolution. may be there are lot more scenarios to consider like column renaming, integrating schema evolution w/ col stats etc. So, we can take it up for 0.12. -- 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]
