Sanil15 opened a new pull request #1152: SAMZA-2319: Simplify Container Allocation logic URL: https://github.com/apache/samza/pull/1152 Container Allocation is a complex part of AM code where we have an opportunity to simplify complex interactions amongst the classes. This can lay grounds for further simplifying the ClusterResourceManager & ClusterProcessManager interactions with ContainerAllocator: Note: I have to follow up this PR with a renaming PR from AbstractContainerAllocator -> ContainerAllocator, I did not do it in the current PR because GIT thinks I am adding stuff to the existing ContainerAllocator.java instead of recognizing renaming the current AbstractContainerAllocator.java class This is a simple refactor, and I need to still add more tests but putting this out for early feedback Adding ContainerPlacements API will become simpler after this
---------------------------------------------------------------- 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
