cshannon commented on code in PR #5359:
URL: https://github.com/apache/accumulo/pull/5359#discussion_r1975429248
##########
core/src/test/java/org/apache/accumulo/core/spi/balancer/HostRegexTableLoadBalancerReconfigurationTest.java:
##########
@@ -108,7 +107,7 @@ public void testConfigurationChanges() {
// getOnlineTabletsForTable
UtilWaitThread.sleep(3000);
this.balance(new
BalanceParamsImpl(Collections.unmodifiableSortedMap(allTabletServers),
- migrations, migrationsOut, DataLevel.USER, tables));
+ migrations, migrationsOut, "USER", tables));
Review Comment:
```suggestion
migrations, migrationsOut, DataLevel.USER.name(), tables));
```
I would still use the enum here for all the tests instead of the hardcoded
string value. this matches what the Manager does as well in this change (it is
just calling .name() on the datalevel)
--
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]
