xiaoyao1991 commented on issue #3227: Revamp RoundRobinPacking default container size determination URL: https://github.com/apache/incubator-heron/pull/3227#issuecomment-476871918 @joshfischer1108 That is true. This puts more burdens on users' knowledge on configuring properly, which is bad. Over the past few months, I've been trying to modify the round robin packing algorithm. In the previous releases (not the latest one), the algorithm basically ignores container-level constraints if all resources required by components exceed the container resource, which might sound flexible, but can be risky if the scheduler (say Aurora) denies the packing plan due to insufficient resource quota. I've made modifications to it so that container-level constraints must be honored if set, but I've seen users complaining about their previously-worked topologies suddenly failing to submit... Trying to find a balance here. What do you think if we reduce the instancedefault cpu to a smaller number, say 0.25?
---------------------------------------------------------------- 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
