keith-turner commented on a change in pull request #1614:
URL: https://github.com/apache/accumulo/pull/1614#discussion_r428065237
##########
File path: test/src/main/java/org/apache/accumulo/test/functional/BulkNewIT.java
##########
@@ -183,6 +184,23 @@ public void testSingleTabletSingleFileOffline() throws
Exception {
}
}
+ @Test
+ public void testMaxTablets() throws Exception {
+ String maxTablets = "0";
+ try (AccumuloClient client =
Accumulo.newClient().from(getClientProps()).build()) {
+ maxTablets = client.instanceOperations().getSystemConfiguration()
+ .get(Property.MASTER_BULK_MAX_TABLETS.getKey());
+
client.instanceOperations().setProperty(Property.MASTER_BULK_MAX_TABLETS.getKey(),
"1");
Review comment:
I am not completely sure, but these two lines may not work as expected
sometimes. What I think may be able to happen is that the property is set on
tserver A by the first line and then the second line reads props from tserver
B. If server B does not clear zoocache, then it could return a stale copy w/o
the prop set on the first line. I am not sure if the client side code to get
props forces a prop cache clear.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
