nsivabalan commented on PR #17724: URL: https://github.com/apache/hudi/pull/17724#issuecomment-4765378229
Rebased onto latest master. PR title and body tightened to match the template. All open feedback addressed: - All yihua suggestions from earlier round (`.withFinalizeWriteParallelism(...)` wiring, getter/field/doc renames, blank-line cleanup) are in. - @danny0405's question about the 512 default: it was the pre-existing `MDT_DEFAULT_PARALLELISM` constant — this PR only makes it user-configurable, it doesn't change the default. Replied inline with the screenshot. CI: one transient Maven artifact lock failure on `test-spark-scala-dml-tests` (Could not acquire shared lock for artifacts: log4j-1.2-api...). Infra flake; should clear on re-run. Ready for review. @yihua @danny0405 PTAL. -- 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]
