jiangxt2 commented on PR #11732: URL: https://github.com/apache/gravitino/pull/11732#issuecomment-4922977068
Rebased onto latest main to resolve conflicts with #11763, and fixed test issues found during local Docker IT runs. Changes: - Regex: added `\s*` between type keyword and `(` in `PARTITION_INFO_PATTERN` to match 3.0+ format where `SHOW CREATE TABLE` outputs a space (e.g. `PARTITION BY LIST (col)`) - Tests: fixed LIST partition tests on 3.0.x/4.0.x — creating a table with empty `PARTITION BY LIST(…)()` is not recognized as partitioned on 3.0+; tests now pass initial assignments at create time - Tests: fixed 1.2.x partition name compatibility (removed hyphens/dots) - Tests: corrected assertion in multi-column LIST test (`partitioning().length` is 1, not 2) Local Docker IT results: - CatalogDorisIT (1.2.x): 19/19 passed - CatalogDoris3xIT (3.0.x): 8/8 passed - CatalogDoris4xIT (4.0.x): 8/8 passed - TestDorisUtils (unit): all passed -- 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]
