ddanielr commented on PR #3551: URL: https://github.com/apache/accumulo/pull/3551#issuecomment-1641493055
> Currently the new test added in (CompactionPriorityQueueMetricsIT.java) fails when run. I believe the check condition is correct and that we may have some unintended behaviors happening when multiple compaction jobs are being submitted to the same queue. > > Also should all references to queue be swapped for compaction group? The jobs still do function in a queue structure. Discussed this with @keith-turner and discovered the issue was with my setup of the tablets and split points. Test was corrected to have more tablets than queue slots and it is now passing. Additional checks were added to validate specific metrics values. #3635 was also created to eventually replace a property that was added for setting a PriorityQueue's maxSize. -- 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]
