[GitHub] [accumulo] DomGarguilo opened a new pull request, #2693: Improvements to MiniAccumuloClusterTest

[GitBox <gi...@apache.org>]

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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii merged pull request #2693: Improvements to MiniAccumuloClusterTest

[GitBox <gi...@apache.org>]

ctubbsii merged PR #2693:
URL: https://github.com/apache/accumulo/pull/2693


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org