You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "EdColeman (via GitHub)" <gi...@apache.org> on 2024/01/30 00:44:50 UTC

[PR] set max files to fix test [accumulo]

EdColeman opened a new pull request, #4205:
URL: https://github.com/apache/accumulo/pull/4205

   The test is failing because as the testErrorDuringCompactionNoOutput test is creating files (50) the default max files limit (15) is exceeded and compaction start.  When the files creation completes, the test was expecting 50 files, but these can be less because of the compactions.  It think this is related to changes in #4127.
   
   Also fixed places where TabletsMetadata was created without try-with-resources.
   
   Also, not sure if the number of files is important.  The test could be changed to accept a number of files >= max files instead of wanting an absolute number, this would reduce the need for an absolute number of files.  The other test was already setting max number of files, so went that way.


-- 
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