This PR did not introduce RebalanceResult nor RebalanceResultImpl. I prefer keeping the current interface and having an internal implementation class. This prevents our external apis from having internal things exposed. For example, in this case, the external api does not need a constructor nor settors. I think it would be good to have more of our external api contract defined by interfaces and try to minimize the number of classes in it.
[ Full content available at: https://github.com/apache/geode/pull/4692 ] This message was relayed via gitbox.apache.org for [email protected]
