Patch looks great - thanks so much, James. Would you mind prefixing the
commit message with "PHOENIX-2256" as that's what ties the pull to the
JIRA? I'll get this committed today.

    James

On Fri, Oct 2, 2015 at 7:34 AM, James Heather <[email protected]>
wrote:

> Hi all (@James T in particular),
>
> I've submitted a pull request to fix the bug I reported in
> https://issues.apache.org/jira/browse/PHOENIX-2256 concerning a failing
> unit test in Java 8.
>
> It was a genuine bug in PMetaDataImpl that just happened to sneak through
> the tests in Java 7 but not Java 8. The traversal order of a collection was
> significant for picking up the bug, and it just happens that it changed
> between Java versions, and the new ordering caused the test to fail.
>
> There was another bug in the same bit of code, which I've also fixed as a
> Brucie bonus, and added a test for, where the eviction policy could cause
> too many things to get evicted unnecessarily.
>
> James
>

Reply via email to