jiangxt2 commented on PR #11916:
URL: https://github.com/apache/gravitino/pull/11916#issuecomment-4913594686
Done. Added IT coverage for set index across all paths:
- `CatalogClickHouseIT`: CREATE + LOAD, ALTER ADD + DELETE, branch coverage
- `CatalogClickHouseClusterIT`: cluster ADD/DELETE with fieldNames
verification
Also found and fixed a DDL bug: ClickHouse requires `set(N)` syntax, not
bare `set`. The unit tests only checked string generation, but the IT tests
caught the actual ClickHouse rejection. Fixed in `ClickHouseTableOperations` +
synced unit test assertions.
Additionally hardened `getClickHouseIndexType` with `startsWith("set(")`
fallback for versions returning parameterized type from
`system.data_skipping_indices`.
--
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]
