You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@samza.apache.org by GitBox <gi...@apache.org> on 2019/09/12 23:57:33 UTC

[GitHub] [samza] Sanil15 opened a new pull request #1152: SAMZA-2319: Simplify Container Allocation logic

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:
users@infra.apache.org


With regards,
Apache Git Services