You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by Josh Elser <jo...@gmail.com> on 2018/09/14 13:11:45 UTC
LoadPlanTest unit test failure on master
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.
Re: LoadPlanTest unit test failure on master
Posted by Josh Elser <jo...@gmail.com>.
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 <jo...@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.
Re: LoadPlanTest unit test failure on master
Posted by Josh Elser <jo...@gmail.com>.
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 <jo...@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.
Re: LoadPlanTest unit test failure on master
Posted by Keith Turner <ke...@deenlo.com>.
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 <jo...@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.
Re: LoadPlanTest unit test failure on master
Posted by Josh Elser <jo...@gmail.com>.
Nevermind, this is silly:
s/TABLET/TABLE/
On 9/14/18 9:11 AM, Josh Elser 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.