Broken test in master: BasicIndexTest

[Maksim Timonin]

Hi, Igniters!

I was discovering how indexes work and found a failed test.
BasicIndexTest#testInlineSizeChange is broken in master and it's not a
flaky case [1]. But it has been failing since 25/09 only.

I discovered that it happened after the IGNITE-13207 ticket merged
(Checkpointer code refactoring) [2]. I'm not sure about the expected
behaviour of the inline index and how checkpointer affects it. But let's
fix it if it is a bug or completely remove this test.

[1]
https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8&testNameId=6131871779633595667&branch=%3Cdefault%3E&tab=testDetails

[2] https://issues.apache.org/jira/browse/IGNITE-13207

[Zhenya Stanilovsky]

Thanks Maxim, the test is correct no need for removal.
I checked 2.9 too, but looks it all ok there. I will take a look.

>Hi, Igniters!
>
>I was discovering how indexes work and found a failed test.
>BasicIndexTest#testInlineSizeChange is broken in master and it's not a
>flaky case [1]. But it has been failing since 25/09 only.
>
>I discovered that it happened after the IGNITE-13207 ticket merged
>(Checkpointer code refactoring) [2]. I'm not sure about the expected
>behaviour of the inline index and how checkpointer affects it. But let's
>fix it if it is a bug or completely remove this test.
>
>[1]
>https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8&testNameId=6131871779633595667&branch=%3Cdefault%3E&tab=testDetails
>
>[2]  https://issues.apache.org/jira/browse/IGNITE-13207
> 

 

[Sergey Chugunov]

Max,

Thanks for spotting this, great catch!

Zhenya, could you please file a ticket of at least Critical priority?

On Fri, Oct 9, 2020 at 9:24 AM Zhenya Stanilovsky
<[hidden email]> wrote:

>
>
> Thanks Maxim, the test is correct no need for removal.
> I checked 2.9 too, but looks it all ok there. I will take a look.
> >Hi, Igniters!
> >
> >I was discovering how indexes work and found a failed test.
> >BasicIndexTest#testInlineSizeChange is broken in master and it's not a
> >flaky case [1]. But it has been failing since 25/09 only.
> >
> >I discovered that it happened after the IGNITE-13207 ticket merged
> >(Checkpointer code refactoring) [2]. I'm not sure about the expected
> >behaviour of the inline index and how checkpointer affects it. But let's
> >fix it if it is a bug or completely remove this test.
> >
> >[1]
> >
> https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8&testNameId=6131871779633595667&branch=%3Cdefault%3E&tab=testDetails
> >
> >[2]  https://issues.apache.org/jira/browse/IGNITE-13207
> >
>

[Anton Kalashnikov]

Hi everyone,

I believe I have a fix for this bug - https://issues.apache.org/jira/browse/IGNITE-13500, So Zhenya you can leave this problem to me.

-- 
Best regards,
Anton Kalashnikov



09.10.2020, 10:19, "Sergey Chugunov" <[hidden email]>:

> Max,
>
> Thanks for spotting this, great catch!
>
> Zhenya, could you please file a ticket of at least Critical priority?
>
> On Fri, Oct 9, 2020 at 9:24 AM Zhenya Stanilovsky
> <[hidden email]> wrote:
>
>>  Thanks Maxim, the test is correct no need for removal.
>>  I checked 2.9 too, but looks it all ok there. I will take a look.
>>  >Hi, Igniters!
>>  >
>>  >I was discovering how indexes work and found a failed test.
>>  >BasicIndexTest#testInlineSizeChange is broken in master and it's not a
>>  >flaky case [1]. But it has been failing since 25/09 only.
>>  >
>>  >I discovered that it happened after the IGNITE-13207 ticket merged
>>  >(Checkpointer code refactoring) [2]. I'm not sure about the expected
>>  >behaviour of the inline index and how checkpointer affects it. But let's
>>  >fix it if it is a bug or completely remove this test.
>>  >
>>  >[1]
>>  >
>>  https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8&testNameId=6131871779633595667&branch=%3Cdefault%3E&tab=testDetails
>>  >
>>  >[2] https://issues.apache.org/jira/browse/IGNITE-13207
>>  >

[Zhenya Stanilovsky]

Of course, i still have no ticket filled. As was discussed in private i will fill other ticket that can lead to potential problems.
 
 

>Hi everyone,
>
>I believe I have a fix for this bug - https://issues.apache.org/jira/browse/IGNITE-13500, So Zhenya you can leave this problem to me.
>
>-- 
>Best regards,
>Anton Kalashnikov
>
>
>
>09.10.2020, 10:19, "Sergey Chugunov" < [hidden email] >:
>> Max,
>>
>> Thanks for spotting this, great catch!
>>
>> Zhenya, could you please file a ticket of at least Critical priority?
>>
>> On Fri, Oct 9, 2020 at 9:24 AM Zhenya Stanilovsky
>> < [hidden email] > wrote:
>>
>>>  Thanks Maxim, the test is correct no need for removal.
>>>  I checked 2.9 too, but looks it all ok there. I will take a look.
>>>  >Hi, Igniters!
>>>  >
>>>  >I was discovering how indexes work and found a failed test.
>>>  >BasicIndexTest#testInlineSizeChange is broken in master and it's not a
>>>  >flaky case [1]. But it has been failing since 25/09 only.
>>>  >
>>>  >I discovered that it happened after the IGNITE-13207 ticket merged
>>>  >(Checkpointer code refactoring) [2]. I'm not sure about the expected
>>>  >behaviour of the inline index and how checkpointer affects it. But let's
>>>  >fix it if it is a bug or completely remove this test.
>>>  >
>>>  >[1]
>>>  >
>>>   https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8&testNameId=6131871779633595667&branch=%3Cdefault%3E&tab=testDetails
>>>  >
>>>  >[2]  https://issues.apache.org/jira/browse/IGNITE-13207
>>>  >