keith-turner opened a new issue, #5917: URL: https://github.com/apache/accumulo/issues/5917
**Describe the bug** VolumeChooserIT.waLogsSentToConfiguredVolumes() can pass w/o verifying write ahead logs were in the expected volume when [this loop] (https://github.com/apache/accumulo/blob/5eb04ebce328d3441b9721d7a548cbcf71b563e1/test/src/main/java/org/apache/accumulo/test/VolumeChooserIT.java#L254) executes zero times. There is variable in the method named `volumesSeens` that is never used also. **Expected behavior** The test actually verifies that write ahead logs are placed on the expected volume. The way the test is using the metadata table to verify this is not a good approach. Write ahead logs are only temporarily in the metadata table (in older versions of Accumulo this was not the case). The test should instead look in zookeepeer and/or HDFS to verify walogs are in the correct volume. -- 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-unsubscr...@accumulo.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
