[MINOR] Move field ordering lookup outside of loop for proto conversion - Githubissues

apache / hudi

Upserts, Deletes And Incremental Processing on Big Data.

https://hudi.apache.org/

Apache License 2.0

5.45k stars 2.42k forks source link

[MINOR] Move field ordering lookup outside of loop for proto conversion #12271

Open the-other-tim-brown opened 4 days ago

the-other-tim-brown commented 4 days ago

Change Logs

Moves call to return ordered fields for lookup outside of the field conversion loop

Impact

Improves performance by avoiding extra lookup calls

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

None

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

hudi-bot commented 4 days ago

CI report:

61d1d5f00abbf463ad2de92dedd7371c4758bb8c Azure: SUCCESS

Bot commands

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