prateekm commented on a change in pull request #1079: SAMZA-2250: Support large job models in standalone. URL: https://github.com/apache/samza/pull/1079#discussion_r296835596
########## File path: samza-core/src/test/java/org/apache/samza/zk/TestZkJobCoordinator.java ########## @@ -62,12 +68,15 @@ import static org.mockito.Mockito.verifyZeroInteractions; import static org.mockito.Mockito.when; +@RunWith(PowerMockRunner.class) Review comment: Possible to refactor to not need this? Prefer not using PowerMock and mocking statics. For example, can you mock the expected metadata store calls instead of the util here? ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
