ctubbsii commented on PR #3479: URL: https://github.com/apache/accumulo/pull/3479#issuecomment-1585195252
> The change updates TabletGroupWatcher to remove servers being shutdown from being sent as candidates for assignment. There was previous logic (e.g. v1.10) that filtered the servers being shutdown from being included in the candidate list. @dtspence Can you please point out where that filtering was occurring? I'm trying to find where the change occurred, to evaluate if there was any intention behind that change, or if it was an accident. Because it's weird that the other current tservers list would now be completely unused and unneeded... it seems like it would have been put there for a reason. Do you know if the change occured in TabletGroupWatcher, or some calling code? Was it broken by #1761? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
