C0urante opened a new pull request, #14005:
URL: https://github.com/apache/kafka/pull/14005

   [Jira](https://issues.apache.org/jira/browse/KAFKA-15177), 
[KIP-875](https://cwiki.apache.org/confluence/display/KAFKA/KIP-875%3A+First-class+offsets+support+in+Kafka+Connect)
   
   
   ### Summary
   Implements the new `alterOffsets` method in all three MM2 connectors. The 
only connector that makes use of the Connect framework-managed offsets at 
runtime is the `MirrorSourceConnector`, but since the other two connectors also 
emit offsets with their records, we also allow users to add/remove offsets for 
them as well, as long as the offsets match the format of the ones that are 
emitted by the connectors.
   
   ### Integration testing
   An integration test case is added (or really, augmented) that focuses 
primarily on the `MirrorSourceConnector`. This case piggybacks off of the 
existing `MirrorConnectorsExactlyOnceIntegrationTest::testReplication` test 
case, which is done for a few reasons:
   - Every new case added to the `MirrorConnectorsIntegrationBaseTest` is run 
by that class and all of its subclasses (of which there are currently five), 
which can lead to blot in testing runtime
   - Graceful shutdown of MM2 tasks may not always occur in a timely fashion, 
which can cause them to emit stale offsets after they are supposed to have been 
stopped; running with exactly-once support eliminates this risk
   - Some subclasses of the `MirrorConnectorsIntegrationBaseTest` 
(specifically, the `MirrorConnectorsIntegrationTransactionsTest` suite) cause 
unpredictable behavior with offsets that can make it harder to verify an exact 
number of records in a replicated topic due to, e.g., control records
   
   Still, this is a little hacky. Happy to change things up if we think it's 
worth the work to find something cleaner.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to