thswlsqls opened a new pull request, #9602:
URL: https://github.com/apache/paimon/pull/9602

   
   ### Purpose
   
   - The description of `clustering.strategy` says the algorithm is chosen from 
the number of columns in `'clustering.by-columns'`, but no such option key 
exists in Paimon.
   - The primary key is `clustering.columns`, with `sink.clustering.by-columns` 
kept only as its fallback. Users following the current text silently get no 
clustering.
   - This completes the rename started in `2558c585c` (added 
`clustering.columns`, made `sink.clustering.by-columns` a fallback) and 
continued in `4e7e4188e` (dropped the deprecated `sink.` prefix but kept the 
old suffix). It does not restore the `sink.` prefix.
   - `docs/generated/core_configuration.html` renders the same string, so both 
files are updated together, matching #9104.
   
   ### Tests
   
   - Description string only, no behavior change — no new tests.
   - `mvn -pl paimon-api clean install` passed (checkstyle, spotless, 184 unit 
tests).
   - `ConfigOptionsDocsCompletenessITCase` in `paimon-docs` passed; reverting 
only the HTML makes it fail on `clustering.strategy`.
   
   
   


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