hudi
hudi copied to clipboard
Fix comment in RFC46
Change Logs
Fix comments in rfc46. You can check all commits.
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist
- [ ] Read through contributor's guide
- [ ] Change Logs and Impact were stated clearly
- [ ] Adequate tests were added if applicable
- [ ] CI passed
@alexeykudinkin Could you please review it? Really thanks! Geng Xiaoyu and me have fix comments in RFC46. I found @DefaultSerializer(HoodieSparkRecordSerializer.class)
and user need not to set spark.kryo.registrator.
https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=11609&view=results
@wzx140 can you please validate that @DefaultSerializer(HoodieSparkRecordSerializer.class)
leads to HoodieSparkRecordSerializer
being used by Kryo when shuffling?
@wzx140 can you please validate that
@DefaultSerializer(HoodieSparkRecordSerializer.class)
leads toHoodieSparkRecordSerializer
being used by Kryo when shuffling?
Yes. I check it in UT with printing some message in HoodieSparkRecordSerializer.
@hudi-bot run azure
@alexeykudinkin Please review the update when you have time.
@hudi-bot run azure
@alexeykudinkin I have implement without the Broadcast and schema inside HoodieSparkRecord and pass record's schema externally. Please review the update when you have time.
@hudi-bot run azure
CI fail with testReadFilterExistAfterBulkInsertPrepped. And it will fix after rebasing on master with commit HUDI-4923.
@hudi-bot run azure
@hudi-bot run azure
@hudi-bot run azure
CI report:
- 0650eb40b10cf97c280809e89f351c15d1c2267f Azure: SUCCESS
Bot commands
@hudi-bot supports the following commands:-
@hudi-bot run azure
re-run the last Azure build