You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "dlmarion (via GitHub)" <gi...@apache.org> on 2025/06/12 13:59:07 UTC
[PR] Don't balance when tservers less than minimum available [accumulo]
dlmarion opened a new pull request, #5638:
URL: https://github.com/apache/accumulo/pull/5638
Closes #5625
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't balance when tservers less than minimum available [accumulo]
Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr commented on PR #5638:
URL: https://github.com/apache/accumulo/pull/5638#issuecomment-2967259869
> I marked this is as draft because I don't think we should merge as-is. There is already a check that the current number of tservers is the same as the last pass to try and account for tserver churn. I'm thinking that, at a minimum, there should be an additional property added (defaults to false) that when enabled enforces the minimum post-startup.
The additional boolean property makes sense. The churn check only needs tservers to be static for a 10 second window. I wonder if that churn window should also be configurable.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't balance when tservers less than minimum available [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #5638:
URL: https://github.com/apache/accumulo/pull/5638#issuecomment-2966902381
I marked this is as draft because I don't think we should merge as-is. There is already a check that the current number of tservers is the same as the last pass to try and account for tserver churn. I'm thinking that, at a minimum, there should be an additional property added (defaults to false) that when enabled enforces the minimum post-startup.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't balance when tservers less than minimum available [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #5638:
URL: https://github.com/apache/accumulo/pull/5638#issuecomment-2967533908
Could also punt on doing assignments and just log a message when there are not enough tservers. Maybe that couldbe done here.
https://github.com/apache/accumulo/blob/cd87901aa21c5634a5b9324729eabd958e9608aa/server/manager/src/main/java/org/apache/accumulo/manager/Manager.java#L1946
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Don't balance when tservers less than minimum available [accumulo]
Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion merged PR #5638:
URL: https://github.com/apache/accumulo/pull/5638
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org