Github user jacksontj commented on the issue:
https://github.com/apache/trafficserver/pull/1088
Re-pasting comments from IRC-- for easier retrieval.
The pop you remove does seem extraneous-- and there are test cases for some
of the refcountcache stuff
(https://github.com/apache/trafficserver/blob/master/iocore/hostdb/test_RefCountCache.cc)
should be easy enough to make a test case for evictions).
As for the priority queue, I ran into a couple issues of it not doing what
I wanted either. Thankfully that has some decent tests-- so assuming you can
figure out the sequence of events that causes the problem-- adding a test case
is quite simple
(https://github.com/apache/trafficserver/blob/master/lib/ts/test_PriorityQueue.cc)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
