[HUDI-7908] Hotfix: HoodieFileGroupReader fails if preCombine and partition fields are the same

[wombatu-kun]

Change Logs

From previous PR:
HoodieFileGroupReader failed if preCombine and partition fields are the same with IllegalArgumentException: Field: ts does not exist in the table schema. precombineField is required but it was filtered from dataSchema as other partition fields.

To fix this I made HoodieFileGroupReaderBasedParquetFileFormat do not filter partitionColumn from dataSchema if it is the same as preCombine field.

But I did it wrong, as you can see from this discussion: https://github.com/apache/hudi/pull/11473/files#r1681098941

There were 2 mistakes:

• filtering condition was wrong during evaluation of dataSchema;
• options did not contain precombineField.

With this PR I fixed it.

Impact

precombineField and partition field may be the same, and it works with local spark and on cluster.

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

none

Documentation Update

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]

CI report:

• 3af1b0c Azure: SUCCESS

Bot commands

@hudi-bot supports the following commands:

• @hudi-bot run azure re-run the last Azure build

[codope]

Thanks for the fix!