DomGarguilo opened a new pull request, #2693: URL: https://github.com/apache/accumulo/pull/2693
I noticed that this test failed after merging in #2685. I'm fairly certain the test failure is unrelated since #2685 only touched JavaScript and FreeMarker files. I'm not too sure why this test failed and have been unable to reproduce the failure but while investigating, I made various changes that seem like improvements to me: * Use unique names for tables (this may have caused the error if it collided with a table name from another test) * Refactor into several new constants and variables * Add resources to try-with-resources blocks * Use NewTableConfiguration to create tables -- 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]
