mynameborat commented on a change in pull request #938: SAMZA-1531: Support run.id in standalone for batch processing. URL: https://github.com/apache/samza/pull/938#discussion_r262815650
########## File path: samza-core/src/main/java/org/apache/samza/coordinator/CoordinationUtils.java ########## @@ -39,6 +42,12 @@ DistributedLockWithState getLockWithState(String lockId); + DistributedReadWriteLock getReadWriteLock(); + + DistributedDataAccess getDataAccess(); + + void setCoordinationSessionListener(CoordinationSessionListener sessionListener); Review comment: This is listener seems very zookeeper specific. Is it possible to push the logic down to implementer and not have this part of coordination utils? ---------------------------------------------------------------- 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
