TheR1sing3un commented on code in PR #7873:
URL: https://github.com/apache/paimon/pull/7873#discussion_r3523446409
##########
paimon-python/pypaimon/write/file_store_commit.py:
##########
@@ -114,10 +114,22 @@ def __init__(self, snapshot_commit: SnapshotCommit,
table, commit_user: str,
self.rollback = CommitRollback(table_rollback) if table_rollback is
not None else None
def commit(self, commit_messages: List[CommitMessage], commit_identifier:
int):
- """Commit the given commit messages in normal append mode."""
+ """Commit write-side messages (data_increment only).
+
+ Compaction results (compact_before / compact_after) must be committed
+ through commit_compact() instead — they produce a separate snapshot
kind
+ and skip row-id assignment.
+ """
if not commit_messages:
return
+ for msg in commit_messages:
+ if not msg.compact_increment.is_empty():
+ raise ValueError(
+ "commit() rejects messages carrying compact_increment; "
+ "use commit_compact() for compaction results."
Review Comment:
fixed
--
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]
