voonhous commented on issue #19996: URL: https://github.com/apache/hudi/issues/19996#issuecomment-5740819820
On the catalog vs session difference: the alias only adds to what exists, so nothing is lost. - Catalog config: with the alias, a catalog that sets `hudi.max-split-size` fills the same `HudiConfig` field as `hudi.target-split-size`, so existing catalogs keep working unchanged. - Session property: `target_split_size` takes its default from that config value, so a catalog configured through the old name also sets the session default. - Trino 483 had no Hudi `max_split_size` session property, so there is no session-level name to alias and no existing session setting breaks. The one behavior change: in Trino 483 the split size was fixed per catalog, while with `hudi-trino` any user can override it for their session through `target_split_size`. That matches how Trino's other connectors handle split sizing, and the 1B minimum rules out zero, but it is a small change in operator control worth a line in the release notes. -- 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]
