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

   ### What changes were proposed in this pull request?
   
   Document the existing ClickHouse catalog contracts for engine-specific table 
properties and data-skipping index expressions.
   
   - Add `graphite.config` to the table property reference and document its 
conditional requirement for `GraphiteMergeTree`.
   - Add `engine_parameters` to the table property reference and document its 
supported engines, load-time restoration, and parameter format.
   - Document that data-skipping indexes with expressions that cannot be 
represented as Gravitino field names are skipped with a warning during table 
loading and are not recreated.
   - Keep the property table columns aligned for readability.
   
   This pull request changes documentation only. It does not modify runtime 
behavior, public APIs, dependencies, or tests.
   
   ### Why are the changes needed?
   
   The ClickHouse catalog already supports `graphite.config` and 
`engine_parameters`, and already skips unsupported index expressions during 
metadata loading. These existing behaviors were not fully described in the 
ClickHouse catalog documentation, making the supported scope and round-trip 
limitations unclear to users.
   
   Fix: N/A (documentation-only correction)
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Users can now find the existing ClickHouse engine property contracts 
and the limitation on expression-based data-skipping indexes in the catalog 
documentation. No runtime behavior or public API changes are introduced.
   
   ### How was this patch tested?
   
   - `./gradlew :docs:build --no-daemon --console=plain` — passed, including 
the repository's OpenAPI lint tasks.
   - `git diff --check` — passed.
   - Cross-checked the documented behavior against the ClickHouse catalog 
implementation and merged pull requests 
[#12274](https://github.com/apache/gravitino/pull/12274) and 
[#12917](https://github.com/apache/gravitino/pull/12917).
   


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