yashmayya commented on PR #13818:
URL: https://github.com/apache/kafka/pull/13818#issuecomment-1578630274

   #### Sink connector offsets alter requests:
   ```
   ConnectorsResource::alterConnectorOffsets -> 
AbstractHerder::alterConnectorOffsets -> 
(Distributed|Standalone)Herder::modifyConnectorOffsets -> 
Worker::alterConnectorOffsets -> Worker::modifySinkConnectorOffsets -> 
Worker::alterSinkConnectorOffset
   ```
   
   #### Source connector offsets alter requests:
   ```
   ConnectorsResource::alterConnectorOffsets -> 
AbstractHerder::alterConnectorOffsets -> 
(Distributed|Standalone)Herder::modifyConnectorOffsets -> 
Worker::alterConnectorOffsets -> Worker::modifySourceConnectorOffsets
   ```
   
   #### Sink connector offsets reset requests:
   ```
   ConnectorsResource::resetConnectorOffsets -> 
AbstractHerder::resetConnectorOffsets -> 
(Distributed|Standalone)Herder::modifyConnectorOffsets -> 
Worker::resetConnectorOffsets -> Worker::modifySinkConnectorOffsets -> 
Worker::resetSinkConnectorOffsets
   ```
   
   #### Source connector offsets reset requests:
   ```
   ConnectorsResource::resetConnectorOffsets -> 
AbstractHerder::resetConnectorOffsets -> 
(Distributed|Standalone)Herder::modifyConnectorOffsets -> 
Worker::resetConnectorOffsets -> Worker::modifySourceConnectorOffsets
   ```
   
   The current flows for altering and resetting offsets along with the use of 
`null` offsets in multiple places to distinguish between alter and reset 
offsets requests might seem a little clunky (especially for sink connectors), 
but I've tried to optimize for code re-use in both the herder implementations 
as well as the worker.


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