phoenix-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Taylor <>
Subject Re: Fixed bug in PMetaDataImpl
Date Fri, 02 Oct 2015 16:02:36 GMT
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.


On Fri, Oct 2, 2015 at 7:34 AM, James Heather <>

> Hi all (@James T in particular),
> I've submitted a pull request to fix the bug I reported in
> 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

View raw message