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

   ## Summary
   
   - `CommitToTable`'s class Javadoc was copy-pasted from `CommitComplete` and 
says the payload "indicates it has completed a commit cycle", which is wrong.
   - `CommitToTable` is a per-table event carrying `tableReference` and 
`snapshotId`; the coordinator sends one per committed table 
(`Coordinator.commitToTable`), while `CommitComplete` is sent once at the end 
of the cycle (`Coordinator.doCommit`).
   - Updated the Javadoc to match the per-table semantics.
   
   ## Testing done
   
   - Docs/comment-only change, no behavior change — no test added. `./gradlew 
:iceberg-kafka-connect:iceberg-kafka-connect-events:spotlessCheck` passed.
   
   ---
   **AI Disclosure**
   - Model: Claude Opus 4.8
   - Platform/Tool: Claude Code
   - Human Oversight: fully reviewed
   - Prompt Summary: Find a contribution in the kafka-connect-events module; 
identified the copy-pasted CommitToTable Javadoc and corrected it to match the 
actual per-table event semantics.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to