EdColeman commented on code in PR #2794:
URL: https://github.com/apache/accumulo/pull/2794#discussion_r910370937
##########
minicluster/src/test/java/org/apache/accumulo/minicluster/MiniAccumuloClusterTest.java:
##########
@@ -210,6 +210,11 @@ public void testPerTableClasspath() throws Exception {
conn.instanceOperations().setProperty(VFS_CONTEXT_CLASSPATH_PROPERTY.getKey() +
"cx1",
jarFile.toURI().toString());
+ // Batchwriter is intermittently failing with a constraint violation
because the TabletServer
+ // is not seeing the property changes above before the BatchWriter below
closes and a
+ // MutationsRejectedException is being thrown.
+ Thread.sleep(5000);
Review Comment:
Checking if the property is set will only verify that the change has
propagated locally. Each process will have its own cache and sets of watchers
on the property node. So checking if the property change has propagated in one
process does not provide any guarantees that all of the other processes have
synced.
--
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]
