So when you iterate through the instances, why can't you just put the heart beat assignments in a separate list and then merge the assignment lists at the end after the scheduler calculates actual assignments.
The interface is confusing because you are returning the same list that is also a parameter to the "schedule" method which is redundant. If you are passing in resultAssignment and adding new assignment to the list, you don't even need to return it. However, I would suggest what I mentioned above and use separate lists and merge them at the end. [ Full content available at: https://github.com/apache/pulsar/pull/2613 ] This message was relayed via gitbox.apache.org for [email protected]
