You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/07/29 17:48:55 UTC

[GitHub] [accumulo] ctubbsii opened a new pull request #1662: Fix FateConcurrencyIT periodic failures

ctubbsii opened a new pull request #1662:
URL: https://github.com/apache/accumulo/pull/1662


   This patch fixes periodic FateConcurrencyIT failures due to insufficient
   threads available for concurrent compactions, caused by the introduction
   of the property
   'tserver.compaction.major.service.default.planner.opts.executors' to
   replace 'tserver.compaction.major.concurrent.max'.
   
   This change now sets a sufficient number of executors to run the test
   consistenly, when one tserver tries to compact 3 or 4 of the 4 total
   tables, and the other is responsible for compacting only 0 or 1.
   
   Additionally:
   
   * Fix a javadoc issue for the improper use of less-than symbol in
     DefaultCompactionPlanner
   * Use a non-deprecated property as an example in MiniAccumuloRunner
     instead of the deprecated 'tserver.compaction.major.concurrent.max'
   * Set instance-wide compaction property only once instead of each time a
     SlowOps object is constructed
   * Inline unneeded private SlowOps constructor
   * Fix typo in FateConcurrencyIT javadoc
   * Use Streams to simplify code logic
   * Avoid creating unused SlowOps in `@Before` method in FateConcurrencyIT
   * Clean up use of TimeUnits and conversions (using nano in an additional
     place, and variables which are named to indicate they are millis)


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii commented on pull request #1662: Fix FateConcurrencyIT periodic failures

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1662:
URL: https://github.com/apache/accumulo/pull/1662#issuecomment-665901844






----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii commented on pull request #1662: Fix FateConcurrencyIT periodic failures

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #1662:
URL: https://github.com/apache/accumulo/pull/1662#issuecomment-665556382






----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii edited a comment on pull request #1662: Fix FateConcurrencyIT periodic failures

Posted by GitBox <gi...@apache.org>.
ctubbsii edited a comment on pull request #1662:
URL: https://github.com/apache/accumulo/pull/1662#issuecomment-665942912






----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii merged pull request #1662: Fix FateConcurrencyIT periodic failures

Posted by GitBox <gi...@apache.org>.
ctubbsii merged pull request #1662:
URL: https://github.com/apache/accumulo/pull/1662


   


----------------------------------------------------------------
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:
users@infra.apache.org