ivakegg commented on code in PR #5070:
URL: https://github.com/apache/accumulo/pull/5070#discussion_r1846657837
##########
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##########
@@ -1091,11 +1110,13 @@ private long balanceTablets() {
}
private List<TabletMigration> checkMigrationSanity(Set<TabletServerId>
current,
- List<TabletMigration> migrations) {
+ List<TabletMigration> migrations, DataLevel level) {
return migrations.stream().filter(m -> {
boolean includeMigration = false;
if (m.getTablet() == null) {
log.error("Balancer gave back a null tablet {}", m);
+ } else if (DataLevel.of(m.getTablet().getTable()) != level) {
+ log.warn("Balancer wants to move a tablet outside of the current
processing level");
Review Comment:
I think this may be too much as we actually can expect the HostRegex
balancer to create migrations outside of the processing level. Should this be
debug or trace maybe?
--
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]
