[HUDI-8519] Fix update with multiple secondary indexes

codope commented 1 week ago

Change Logs

Multiple secondary indexes (or functional index) exist in different partitions but still we use the same file id prefix. So, thre is a chance of collision in the append handle when two different secondary index have same file id prefix and same shard. This PR fixes the file id prefix in such a case. Added updates to the existing test case which creates multiple secondary index.

Impact

Fix updates with multiple secondary indexes.

Risk level (write none, low medium or high below)

low

only affects sec index and func index.

Documentation Update

Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the ticket number here and follow the instruction to make changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

nsivabalan commented 1 week ago

can we write tests for functional index as well since the fix is applicable for FI as well

hudi-bot commented 1 week ago

CI report:

ba8dd8d9fcc6281f50cf677674d425aea2c75f7a Azure: SUCCESS

Bot commands

@hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build

codope commented 6 days ago

Closing in favor of #12263

apache / hudi