You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "ArnavBalyan (via GitHub)" <gi...@apache.org> on 2026/05/31 15:46:01 UTC
[PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
ArnavBalyan opened a new pull request, #8057:
URL: https://github.com/apache/paimon/pull/8057
### Purpose
- Fix typo in the readme, add more detailed examples for unsupported modes in pk-clustering-override.
- The behaviour for unsupported config is not clear today, demonstrate it with an exmaple.
### Tests
- NA/Docs
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8057:
URL: https://github.com/apache/paimon/pull/8057
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on code in PR #8057:
URL: https://github.com/apache/paimon/pull/8057#discussion_r3331218722
##########
docs/docs/primary-key-table/pk-clustering-override.md:
##########
@@ -97,4 +97,20 @@ values, so queries like `WHERE id = 12345` remain efficient.
- Merge engine: `partial-update` or `aggregation`.
- Changelog producer: `lookup` or `full-compaction`.
-- Configue: `sequence.fields` or `record-level.expire-time`.
+- Configuration: `sequence.fields` or `record-level.expire-time`.
+
+For example, the following combination is rejected at table creation:
Review Comment:
We don't need this example for **Unsupported modes**.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8057:
URL: https://github.com/apache/paimon/pull/8057#issuecomment-4637978792
I reviewed this PR as part of a pass over recent open non-draft Paimon PRs. I did not find any clear correctness, compatibility, or regression issue to flag from the current diff.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "ArnavBalyan (via GitHub)" <gi...@apache.org>.
ArnavBalyan commented on PR #8057:
URL: https://github.com/apache/paimon/pull/8057#issuecomment-4588820995
Got it updated ty!
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "ArnavBalyan (via GitHub)" <gi...@apache.org>.
ArnavBalyan commented on code in PR #8057:
URL: https://github.com/apache/paimon/pull/8057#discussion_r3331227626
##########
docs/docs/primary-key-table/pk-clustering-override.md:
##########
@@ -97,4 +97,20 @@ values, so queries like `WHERE id = 12345` remain efficient.
- Merge engine: `partial-update` or `aggregation`.
- Changelog producer: `lookup` or `full-compaction`.
-- Configue: `sequence.fields` or `record-level.expire-time`.
+- Configuration: `sequence.fields` or `record-level.expire-time`.
+
+For example, the following combination is rejected at table creation:
Review Comment:
Got it updated ty!
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "ArnavBalyan (via GitHub)" <gi...@apache.org>.
ArnavBalyan commented on PR #8057:
URL: https://github.com/apache/paimon/pull/8057#issuecomment-4618810770
Thanks @JingsongLi! I'll resolve the conflict shortly
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [docs] Fix typo and add unsupported example for unsupported config [paimon]
Posted by "ArnavBalyan (via GitHub)" <gi...@apache.org>.
ArnavBalyan commented on PR #8057:
URL: https://github.com/apache/paimon/pull/8057#issuecomment-4628249501
Conflict resolve thanks!
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org