yuqi1129 opened a new pull request, #11018:
URL: https://github.com/apache/gravitino/pull/11018

   ### What changes were proposed in this pull request?
   
   Support ClickHouse table creation with Gravitino date partition transforms:
   
   - `Transforms.year(column)` -> `PARTITION BY toYear(column)`
   - `Transforms.month(column)` -> `PARTITION BY toYYYYMM(column)`
   - `Transforms.day(column)` -> `PARTITION BY toDate(column)`
   
   This PR also removes the stale identity-only helper in 
`ClickHouseTableOperations` and uses the shared ClickHouse SQL utility path for 
partition expression rendering.
   
   ### Why are the changes needed?
   
   ClickHouse catalog table creation previously only supported identity 
partitioning, while loading existing ClickHouse tables already recognized 
`toYear`, `toYYYYMM`, and `toDate` partition expressions.
   
   Fix: #11017
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Users can now create ClickHouse MergeTree-family tables with `year`, 
`month`, and `day` partition transforms through Gravitino.
   
   ### How was this patch tested?
   
   ```bash
   ./gradlew :catalogs-contrib:catalog-jdbc-clickhouse:spotlessApply
   ./gradlew :catalogs-contrib:catalog-jdbc-clickhouse:test --tests 
org.apache.gravitino.catalog.clickhouse.operations.TestClickHouseTableOperations
 -PskipITs
   ./gradlew :catalogs-contrib:catalog-jdbc-clickhouse:test --tests 
org.apache.gravitino.catalog.clickhouse.integration.test.CatalogClickHouseIT.testCreateAndLoadWithPartitionSortAndIndexes
 -PskipTests -PskipDockerTests=false
   ```


-- 
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]

Reply via email to