IGNITE-12508

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

IGNITE-12508

Atri Sharma-2
Hi all,

I have raised a PR for IGNITE-12508:

https://github.com/apache/ignite/pull/8802

Requesting inputs and feedback on the same, please.

Regards,

Atri

--
Regards,

Atri
Apache Concerted
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Ilya Kasnacheev
Hello!

I have left some comments to the PR.

Regards,
--
Ilya Kasnacheev


пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:

> Hi all,
>
> I have raised a PR for IGNITE-12508:
>
> https://github.com/apache/ignite/pull/8802
>
> Requesting inputs and feedback on the same, please.
>
> Regards,
>
> Atri
>
> --
> Regards,
>
> Atri
> Apache Concerted
>
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Atri Sharma-2
Thank you! I have updated the PR per your comments.

On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
<[hidden email]> wrote:

>
> Hello!
>
> I have left some comments to the PR.
>
> Regards,
> --
> Ilya Kasnacheev
>
>
> пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
>
> > Hi all,
> >
> > I have raised a PR for IGNITE-12508:
> >
> > https://github.com/apache/ignite/pull/8802
> >
> > Requesting inputs and feedback on the same, please.
> >
> > Regards,
> >
> > Atri
> >
> > --
> > Regards,
> >
> > Atri
> > Apache Concerted
> >

--
Regards,

Atri
Apache Concerted
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Ilya Kasnacheev
Hello!

Thank you! I have started a MTCGA run since there were not any.

Regards,
--
Ilya Kasnacheev


ср, 17 февр. 2021 г. в 07:49, Atri Sharma <[hidden email]>:

> Thank you! I have updated the PR per your comments.
>
> On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
> <[hidden email]> wrote:
> >
> > Hello!
> >
> > I have left some comments to the PR.
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
> >
> > > Hi all,
> > >
> > > I have raised a PR for IGNITE-12508:
> > >
> > > https://github.com/apache/ignite/pull/8802
> > >
> > > Requesting inputs and feedback on the same, please.
> > >
> > > Regards,
> > >
> > > Atri
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > Apache Concerted
> > >
>
> --
> Regards,
>
> Atri
> Apache Concerted
>
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Atri Sharma-2
Hello,

Thanks for helping with this. According to the bot's comment on JIRA,
the tests that are failing are:
JdbcThinJdbcToCacheDataTypesCoverageTest.testTimeDataType
IgnitePdsWithIndexingCoreTestSuite:
WalRecoveryTxLogicalRecordsTest.testRecoveryNoPageLost3

I have verified that these tests pass locally for me (beasted them)
and also manually debugged to see if the new method was giving the
correct behaviour.

Please advice.

Atri

On Wed, Feb 17, 2021 at 2:37 PM Ilya Kasnacheev
<[hidden email]> wrote:

>
> Hello!
>
> Thank you! I have started a MTCGA run since there were not any.
>
> Regards,
> --
> Ilya Kasnacheev
>
>
> ср, 17 февр. 2021 г. в 07:49, Atri Sharma <[hidden email]>:
>
> > Thank you! I have updated the PR per your comments.
> >
> > On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
> > <[hidden email]> wrote:
> > >
> > > Hello!
> > >
> > > I have left some comments to the PR.
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
> > >
> > > > Hi all,
> > > >
> > > > I have raised a PR for IGNITE-12508:
> > > >
> > > > https://github.com/apache/ignite/pull/8802
> > > >
> > > > Requesting inputs and feedback on the same, please.
> > > >
> > > > Regards,
> > > >
> > > > Atri
> > > >
> > > > --
> > > > Regards,
> > > >
> > > > Atri
> > > > Apache Concerted
> > > >
> >
> > --
> > Regards,
> >
> > Atri
> > Apache Concerted
> >

--
Regards,

Atri
Apache Concerted
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Ilya Kasnacheev
Hello!

I think these tests are flaky, I have triggered them to re-run.

Regards,
--
Ilya Kasnacheev


ср, 17 февр. 2021 г. в 18:19, Atri Sharma <[hidden email]>:

> Hello,
>
> Thanks for helping with this. According to the bot's comment on JIRA,
> the tests that are failing are:
> JdbcThinJdbcToCacheDataTypesCoverageTest.testTimeDataType
> IgnitePdsWithIndexingCoreTestSuite:
> WalRecoveryTxLogicalRecordsTest.testRecoveryNoPageLost3
>
> I have verified that these tests pass locally for me (beasted them)
> and also manually debugged to see if the new method was giving the
> correct behaviour.
>
> Please advice.
>
> Atri
>
> On Wed, Feb 17, 2021 at 2:37 PM Ilya Kasnacheev
> <[hidden email]> wrote:
> >
> > Hello!
> >
> > Thank you! I have started a MTCGA run since there were not any.
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > ср, 17 февр. 2021 г. в 07:49, Atri Sharma <[hidden email]>:
> >
> > > Thank you! I have updated the PR per your comments.
> > >
> > > On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
> > > <[hidden email]> wrote:
> > > >
> > > > Hello!
> > > >
> > > > I have left some comments to the PR.
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
> > > >
> > > > > Hi all,
> > > > >
> > > > > I have raised a PR for IGNITE-12508:
> > > > >
> > > > > https://github.com/apache/ignite/pull/8802
> > > > >
> > > > > Requesting inputs and feedback on the same, please.
> > > > >
> > > > > Regards,
> > > > >
> > > > > Atri
> > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Atri
> > > > > Apache Concerted
> > > > >
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > Apache Concerted
> > >
>
> --
> Regards,
>
> Atri
> Apache Concerted
>
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Atri Sharma-2
Thank you!

Where do I figure out which tests are currently flaky?

On Wed, Feb 17, 2021 at 8:58 PM Ilya Kasnacheev
<[hidden email]> wrote:

>
> Hello!
>
> I think these tests are flaky, I have triggered them to re-run.
>
> Regards,
> --
> Ilya Kasnacheev
>
>
> ср, 17 февр. 2021 г. в 18:19, Atri Sharma <[hidden email]>:
>
> > Hello,
> >
> > Thanks for helping with this. According to the bot's comment on JIRA,
> > the tests that are failing are:
> > JdbcThinJdbcToCacheDataTypesCoverageTest.testTimeDataType
> > IgnitePdsWithIndexingCoreTestSuite:
> > WalRecoveryTxLogicalRecordsTest.testRecoveryNoPageLost3
> >
> > I have verified that these tests pass locally for me (beasted them)
> > and also manually debugged to see if the new method was giving the
> > correct behaviour.
> >
> > Please advice.
> >
> > Atri
> >
> > On Wed, Feb 17, 2021 at 2:37 PM Ilya Kasnacheev
> > <[hidden email]> wrote:
> > >
> > > Hello!
> > >
> > > Thank you! I have started a MTCGA run since there were not any.
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > ср, 17 февр. 2021 г. в 07:49, Atri Sharma <[hidden email]>:
> > >
> > > > Thank you! I have updated the PR per your comments.
> > > >
> > > > On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
> > > > <[hidden email]> wrote:
> > > > >
> > > > > Hello!
> > > > >
> > > > > I have left some comments to the PR.
> > > > >
> > > > > Regards,
> > > > > --
> > > > > Ilya Kasnacheev
> > > > >
> > > > >
> > > > > пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
> > > > >
> > > > > > Hi all,
> > > > > >
> > > > > > I have raised a PR for IGNITE-12508:
> > > > > >
> > > > > > https://github.com/apache/ignite/pull/8802
> > > > > >
> > > > > > Requesting inputs and feedback on the same, please.
> > > > > >
> > > > > > Regards,
> > > > > >
> > > > > > Atri
> > > > > >
> > > > > > --
> > > > > > Regards,
> > > > > >
> > > > > > Atri
> > > > > > Apache Concerted
> > > > > >
> > > >
> > > > --
> > > > Regards,
> > > >
> > > > Atri
> > > > Apache Concerted
> > > >
> >
> > --
> > Regards,
> >
> > Atri
> > Apache Concerted
> >

--
Regards,

Atri
Apache Concerted
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-12508

Ilya Kasnacheev
Hello!

MTCGA will do that for 90% of tests but you may need to re-run a few suites
from its interface.
https://mtcga.gridgain.com/pr.html?serverId=apache&suiteId=IgniteTests24Java8_RunAll&branchForTc=pull/8802/head&action=Latest

Regards,
--
Ilya Kasnacheev


ср, 17 февр. 2021 г. в 18:29, Atri Sharma <[hidden email]>:

> Thank you!
>
> Where do I figure out which tests are currently flaky?
>
> On Wed, Feb 17, 2021 at 8:58 PM Ilya Kasnacheev
> <[hidden email]> wrote:
> >
> > Hello!
> >
> > I think these tests are flaky, I have triggered them to re-run.
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > ср, 17 февр. 2021 г. в 18:19, Atri Sharma <[hidden email]>:
> >
> > > Hello,
> > >
> > > Thanks for helping with this. According to the bot's comment on JIRA,
> > > the tests that are failing are:
> > > JdbcThinJdbcToCacheDataTypesCoverageTest.testTimeDataType
> > > IgnitePdsWithIndexingCoreTestSuite:
> > > WalRecoveryTxLogicalRecordsTest.testRecoveryNoPageLost3
> > >
> > > I have verified that these tests pass locally for me (beasted them)
> > > and also manually debugged to see if the new method was giving the
> > > correct behaviour.
> > >
> > > Please advice.
> > >
> > > Atri
> > >
> > > On Wed, Feb 17, 2021 at 2:37 PM Ilya Kasnacheev
> > > <[hidden email]> wrote:
> > > >
> > > > Hello!
> > > >
> > > > Thank you! I have started a MTCGA run since there were not any.
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > ср, 17 февр. 2021 г. в 07:49, Atri Sharma <[hidden email]>:
> > > >
> > > > > Thank you! I have updated the PR per your comments.
> > > > >
> > > > > On Tue, Feb 16, 2021 at 9:29 PM Ilya Kasnacheev
> > > > > <[hidden email]> wrote:
> > > > > >
> > > > > > Hello!
> > > > > >
> > > > > > I have left some comments to the PR.
> > > > > >
> > > > > > Regards,
> > > > > > --
> > > > > > Ilya Kasnacheev
> > > > > >
> > > > > >
> > > > > > пн, 15 февр. 2021 г. в 18:55, Atri Sharma <[hidden email]>:
> > > > > >
> > > > > > > Hi all,
> > > > > > >
> > > > > > > I have raised a PR for IGNITE-12508:
> > > > > > >
> > > > > > > https://github.com/apache/ignite/pull/8802
> > > > > > >
> > > > > > > Requesting inputs and feedback on the same, please.
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > Atri
> > > > > > >
> > > > > > > --
> > > > > > > Regards,
> > > > > > >
> > > > > > > Atri
> > > > > > > Apache Concerted
> > > > > > >
> > > > >
> > > > > --
> > > > > Regards,
> > > > >
> > > > > Atri
> > > > > Apache Concerted
> > > > >
> > >
> > > --
> > > Regards,
> > >
> > > Atri
> > > Apache Concerted
> > >
>
> --
> Regards,
>
> Atri
> Apache Concerted
>