agresch opened a new pull request, #3633: URL: https://github.com/apache/storm/pull/3633
## What is the purpose of the change The scheduler has a bug where acker memory is not considered in the scheduling estimate. The case I found was where a topology should fit on two supervisors, but the cluster has 1 available and 2 blacklisted. The scheduler thinks the topology should fit on one supervisor and fails to schedule, but also fails to release a supervisor from the blacklist, resulting in the topology never getting scheduled. With this fix, the scheduler properly detects the topology will need to be scheduled on two supervisors and releases one from the blacklist and schedules successfully. Switched some scheduling logs from trace to debug to make debugging scheduling issues easier. ## How was the change tested Added a unit test to validate the topology will properly schedule after releasing a supervisor from the blacklist. The fix was also validated on our internal cluster integration tests. -- 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. To unsubscribe, e-mail: dev-unsubscr...@storm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
