[PR] Update sharding document [shardingsphere]

["zhaojinchao95 (via GitHub)" <gi...@apache.org>]

zhaojinchao95 opened a new pull request, #38615:
URL: https://github.com/apache/shardingsphere/pull/38615

   Fixes #38178 


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Update sharding document [shardingsphere]

["zhaojinchao95 (via GitHub)" <gi...@apache.org>]

zhaojinchao95 merged PR #38615:
URL: https://github.com/apache/shardingsphere/pull/38615


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Update sharding document [shardingsphere]

["zhaojinchao95 (via GitHub)" <gi...@apache.org>]

zhaojinchao95 commented on PR #38615:
URL: https://github.com/apache/shardingsphere/pull/38615#issuecomment-4242949506

    It can be submitted now.
   
     Per your latest instruction, if we ignore the unrelated TODO removal in ShardingTable.java, the main blocker for this PR has been resolved:
   
     - The remaining outdated keyGenerateStrategy: examples under docs/document/content have been fully updated.
     - reference/management/_index.* now clearly describes persisted metadata nodes in the registry center, so it no longer gets mixed up with user-facing configuration format.
     - The current working tree only contains 6 documentation files, and git diff --name-only shows a clean, scoped change set.
   
     What I verified:
   
     - rg -n "keyGenerateStrategy:" docs/document/content -S now returns no results.
     - The current diff only includes these 6 documentation files:
         - docs/document/content/reference/management/_index.cn.md
         - docs/document/content/reference/management/_index.en.md
         - docs/document/content/user-manual/common-config/builtin-algorithm/expr.cn.md
         - docs/document/content/user-manual/common-config/builtin-algorithm/expr.en.md
         - docs/document/content/user-manual/common-config/builtin-algorithm/sharding.cn.md
         - docs/document/content/user-manual/common-config/builtin-algorithm/sharding.en.md
   
     One note:
   
     - I did not run Maven, Spotless, or docs-site rendering checks. Since this is a docs-only change, the risk is low, but if you want a stricter handoff, a minimal verification step would still be reasonable.
   
     If you want, I can also help draft a suitable commit message next.
   


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org