keith-turner commented on issue #3466:
URL: https://github.com/apache/accumulo/issues/3466#issuecomment-1927798328

   > Curious if you evaluated Guava 33.0's MinMaxPriorityQueue when you created 
CompactionJobPriorityQueue.
   
   Yeah I did.  One reason I did not use was the following comment on the 
javadoc.  So when trying to use it as a bounded size prio Q it seems like it 
would be slow. 
   
   > If you only access one end of the queue, and don't use a maximum size, 
this class is functionally equivalent to 
[PriorityQueue](https://docs.oracle.com/javase/9/docs/api/java/util/PriorityQueue.html?is-external=true),
 but significantly slower.
   
   A second reason I did not use was to because when a new jobs are added for 
tablet, need to efficiently remove any previous jobs related to the tablet that 
happen to be in the queue.


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

Reply via email to