Davis-Zhang-Onehouse opened a new pull request, #13594: URL: https://github.com/apache/hudi/pull/13594
### Change Logs We only allow column with Integral types (int, short int, long, bigint, etc), string and timestamp types for SI due to limitations that secondary index store and match column values based on string representations, for other column types there can be surprises which leads to unexpected outcome. For example, double value 20 tostring is 20.0. If we look up for 20, we have to give 20.0 (double value), give 20(int) it results in a different string representations and cause index corruption or look up correctness issue. ### Impact Draw a line to exclude trick column types and leave it for future design recap to support those types. ### Risk level (write none, low medium or high below) none ### Documentation Update Need to make it clear on the column type we support in user facing doc. ### Contributor's checklist - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute) - [ ] Change Logs and Impact were stated clearly - [ ] Adequate tests were added if applicable - [ ] CI passed -- 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]
