accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Josh Elser <josh.el...@gmail.com>
Subject Re: LoadPlanTest unit test failure on master
Date Fri, 14 Sep 2018 14:37:53 GMT
Nevermind, Christopher has already fixed this it seems..

On 9/14/18 10:10 AM, Josh Elser wrote:
> I'll tag you in a PR. Trivial fix. Don't sweat it :)
> 
> On 9/14/18 10:09 AM, Keith Turner wrote:
>> I will look into it.  This branch was outstanding for a long time.
>> Yesterday I merged it, resolved conflicts, and then only ran mvn
>> compile.  I should have ran mvn verify again and waited.
>>
>> On Fri, Sep 14, 2018 at 9:11 AM, Josh Elser <josh.elser@gmail.com> wrote:
>>> Color me surprised: this fails for me out of the box on
>>> 7ef140ec40c3768859b848350db8c6d6d20f7a56
>>>
>>> [INFO] Running org.apache.accumulo.core.data.LoadPlanTest
>>> [ERROR] Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed:
>>> 0.168 s <<< FAILURE! - in org.apache.accumulo.core.data.LoadPlanTest
>>> [ERROR] testTypes(org.apache.accumulo.core.data.LoadPlanTest)  Time 
>>> elapsed:
>>> 0.074 s  <<< FAILURE!
>>> java.lang.AssertionError: expected:<[f5.rf:TABLET:yyy:null,
>>> f3.rf:DATA:368:479, f2.rf:DATA:abc:def, f7.rf:TABLET:www:null,
>>> f9.rf:TABLET:xxx:null, fb.rf:TABLET:heg:klt, fd.rf:TABLET:null:null,
>>> f6.rf:TABLET:null:bbb, fc.rf:TABLET:agt:ctt, f8.rf:TABLET:null:ccc,
>>> f1.rf:DATA:1112:1145, f4.rf:TABLET:null:aaa, fa.rf:TABLET:1138:1147]> 
>>> but
>>> was:<[f5.rf:TABLE:yyy:null, f4.rf:TABLE:null:aaa, f9.rf:TABLE:xxx:null,
>>> f2.rf:FILE:abc:def, f3.rf:FILE:368:479, f7.rf:TABLE:www:null,
>>> f8.rf:TABLE:null:ccc, fa.rf:TABLE:1138:1147, fd.rf:TABLE:null:null,
>>> f1.rf:FILE:1112:1145, fc.rf:TABLE:agt:ctt, f6.rf:TABLE:null:bbb,
>>> fb.rf:TABLE:heg:klt]>
>>>          at
>>> org.apache.accumulo.core.data.LoadPlanTest.testTypes(LoadPlanTest.java:93) 
>>>
>>>
>>> @Keith, I see that HEAD is a commit which touches LoadPlanTest. 
>>> Haven't yet
>>> dug into the test (seems to be new code since I've touched Accumulo),
>>> lobbing this as a softball for now and will investigate as time allows.

Mime
View raw message