ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexey Goncharuk <alexey.goncha...@gmail.com>
Subject Re: IGNITE-6827 - Review needed.
Date Mon, 02 Apr 2018 09:48:18 GMT
Guys, especially Vladimir Ozerov, Yakov Zhanov, Dmitriy Setrakyan,

While the technical side of the change is being reviewed, I would also ask
you to pay attention to the public API changes here.

--AG

2018-03-21 23:43 GMT+03:00 Pavel Tupitsyn <ptupitsyn@apache.org>:

> Hi Alexei,
>
> >  Pavel Tupitsyn, could you lend a hand to help completing .NET part (two
> new
> public API methods are added as a part of working on [3], java code in
> branch [2])
>
> Please file a separate ticket for .NET changes.
> I would like to help, but maybe some time later. No need to delay your
> changes because of that.
>
> Thanks,
> Pavel
>
> On Wed, Mar 21, 2018 at 1:08 PM, Dmitry Pavlov <dpavlov.spb@gmail.com>
> wrote:
>
> > Hi Alexei ,
> >
> > Thank you for contribution, unfortunately there is too much tests failed
> > there:
> > .NET is broken
> >
> >   Ignite Platform .NET Long Running [ tests 3 ]
> >     CachePartitionedNearEnabledTest.TestTransactionScopeMultiCache(True)
> >     CachePartitionedTest.TestTransactionScopeMultiCache(True)
> >     CacheReplicatedTest.TestTransactionScopeMultiCache(True)
> >
> >    Ignite Platform .NET Core Linux [ tests 2 TC_EXIT_CODE ]
> >      TransactionsParityTest.TestTransactions (fail rate 0,0%)
> >      CachePartitionedTest.TestTransactionScopeMultiCache(True) (fail
> rate
> > 0,0%)
> >
> >    Ignite Platform .NET [ tests 1 ] agoncharuk More info >>
> >     Test fail rate less than 1%, probably new failure  exe:
> > TransactionsParityTest.TestTransactions (fail rate 0,0%)
> >
> > Too much timeouts:
> >    Ignite Cache Failover [2]
> >  with CacheAsyncOperationsFailoverTxTest.testPutAllAsyncFailover (last
> > started)
> >
> >    ~[Obsolete] Ignite Cache Expiry Policy [ tests 1 ]
> > with  IgniteCacheExpiryPolicyTestSuite:
> > IgniteCacheTtlCleanupSelfTest.testDeferredDeleteTtl
> >
> >  Ignite Cache Restarts [2]
> >   with
> > GridCachePutAllFailoverSelfTest.testPutAllFailoverNearDisabled
> ThreeBackups
> > (last started)
> >
> >   Ignite Cache Restarts [1]
> >    with
> > GridCachePartitionedNearDisabledOptimisticTxNodeRestartTest.
> > testRestartWithTxTenNodesTwoBackups
> > (last started)
> >
> >  Cache [6]
> >    with  WalModeChangeAdvancedSelfTest.testServerRestartNonCoordinator
> > (last started)
> > it is not all.
> >
> > Please fix tests before review. Also you could change ticket status to
> 'In
> > Progress'.
> >
> > As easy option, you can merge current master to your branch, it is
> probably
> > that some test were fixed already.
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > ср, 21 мар. 2018 г. в 9:58, Alexei Scherbakov <
> > alexey.scherbakoff@gmail.com
> > >:
> >
> > > Igniters,
> > >
> > > I prepared rather important patch [1] related to grid stability and
> need
> > a
> > > review and some help to finish it.
> > >
> > > Semyon Boikov, could you please take a look on patch code ? [2]
> > >
> > > Pavel Tupitsyn, could you lend a hand to help completing .NET part (two
> > new
> > > public API methods are added as a part of working on [3], java code in
> > > branch [2])
> > >
> > > [1] https://issues.apache.org/jira/browse/IGNITE-6827
> > >
> > > [2] https://github.com/gridgain/apache-ignite/tree/ignite-6827-2
> > >
> > > [3] https://issues.apache.org/jira/browse/IGNITE-7910
> > >
> > > --
> > >
> > > Best regards,
> > > Alexei Scherbakov
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message