dnishimura commented on a change in pull request #1016: SAMZA-2179: Move the 
StartpointVisitor abstraction to SystemAdmin interface.
URL: https://github.com/apache/samza/pull/1016#discussion_r279829110
 
 

 ##########
 File path: samza-api/src/main/java/org/apache/samza/system/SystemAdmin.java
 ##########
 @@ -164,4 +165,10 @@ default void deleteMessages(Map<SystemStreamPartition, 
String> offsets) {
     throw new UnsupportedOperationException();
   }
 
+  /**
+   * Builds and returns the {@link StartpointVisitor} for a system.
+   */
+  default StartpointVisitor getStartpointVisitor() {
 
 Review comment:
   Are you going to integrate this with the `OffsetManager` in a follow PR? I 
don't see it being used in the current PR.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to