prashantwason opened a new pull request, #8494: URL: https://github.com/apache/hudi/pull/8494
[HUDI-6099] Improved the performance of checking for valid commits when tagging record location. ### Change Logs 1. Moved the checkIfValidCommit function to HoodieIndexUtils so that it can be shared across various indexes. 2. checkIfValidCommit not accepts a HoodieTimeline instead of HoodieTableMetaClient. Hence, the timeline does not need to be computed for checking each and every record. 3. Fixes HoodieInMemoryHashIndex which was not checking for valid commit when tagging location. ### Impact Improved performance when tagging a large number of records. ### Risk level (write none, low medium or high below) None. Basically a code reorg. ### Documentation Update None ### 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]
