taowenjun commented on PR #13085: URL: https://github.com/apache/seatunnel/pull/13085#issuecomment-6600454766
@SEZ9 The documentation clarification requested in #13087 is now included in this PR, in both English and Chinese. The current head is `3b88243724b7b0d84acb2e798f93f5ec217d201b` (rebased onto dev). The docs now explicitly state that `use_table_compression` is ignored for non-ORC tables and skipped when the table does not exist at sink initialization, including `orc.compress` declared in `save_mode_create_template`. Enabling this option alone does not fail either case; explicit `compress_codec`, existing defaults, and normal validation and table-creation errors still apply. - [English documentation](https://github.com/taowenjun/seatunnel/blob/3b88243724b7b0d84acb2e798f93f5ec217d201b/docs/en/connectors/sink/Hive.md#use_table_compression-boolean) - [Chinese documentation](https://github.com/taowenjun/seatunnel/blob/3b88243724b7b0d84acb2e798f93f5ec217d201b/docs/zh/connectors/sink/Hive.md#use_table_compression-boolean) This PR remains limited to compression inheritance for existing ORC tables. Parquet and other formats will be separate follow-up work. The PR description also retains the caveat that the referenced Docker E2E matrices used a JDK 11 engine container and do not establish JDK 17 engine runtime coverage. The clarification is ready for review together with the implementation and tests. -- 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]
