voonhous commented on issue #19666: URL: https://github.com/apache/hudi/issues/19666#issuecomment-5394515221
Update from the #19648 review: the write side of this is now handled there. `HoodieWriterUtils#validateTableConfig` rejects slash partitioning with more than one partition field (df.write and SQL writes, new and existing tables), and `HoodieCatalogTable#extraTableConfig` rejects it at CREATE TABLE -- the suggested fix above, pulled forward because #19648's CCE fix would otherwise have made the broken layout silently writable through the row writer. What remains for this issue is the legacy read side: tables that already hold a multi-field slash layout still throw (`CustomKeyGenerator`) or silently return zero rows on partition-filtered queries (`SimpleKeyGenerator`/`ComplexKeyGenerator`), and registering such a table via `CREATE TABLE ... location` now fails the same validation. Whether to offer a migration/read path for them, or document recreate-as-single-field as the answer, is the open question. -- 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]
