Davis-Zhang-Onehouse opened a new pull request, #13622: URL: https://github.com/apache/hudi/pull/13622
### Change Logs 2. Bug fix for getSecondaryIndexRecords API. For V1 it is the correct behavior that we return seckey -> reckey mapping yet in v2 we reversed it. 4. To support the bug fix, we found that because the seckey can be null, running operations like reduce by key or group by key they will error out. So relevant APIs that handle pair data with null is revised. 5. To handle null value, we revise the interface to return HoodiePairData<String, String>, instead of HotoodiePairData<String, Set<String>> 6. To handle hoodiePairData with null key, we revised API dedupeAndCollectAsMap and introduced dedupeAndCollectAsMapOfSet to serve use cases where we need to convert HoodiePairData to Map<> 7. When devising the APIs we found that when multiple actions like collectAsList is invoked on HoodieListPairData, we hit stream is closed error. Extend it to handle such case via persist() turning lazy evaluation to an eager one and cache the data. Unpersist will remove the cached data. ### Impact bug free for index lookup API, along with a few util class enhancement supporting that ### Risk level (write none, low medium or high below) none ### 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]
