Dmitry, the fix looks good to me, I would appreciate if you merge it. Thanks, Pavel
On Tue, May 29, 2018 at 12:02 AM, Dmitry Pavlov <dpavlov....@gmail.com> wrote: > Hi Pavel, > > Thank you for pointing to this. We've noticed that, and Ilya K. has already > prepared the fix, > https://issues.apache.org/jira/browse/IGNITE-8604 > https://github.com/apache/ignite/pull/4072 > > If you have some time you can apply the PR, or I will merge it later. > > Sincerely, > Dmitriy Pavlov > > пн, 28 мая 2018 г. в 23:45, Pavel Tupitsyn <ptupit...@apache.org>: > > > Hi Dmirty, > > > > IGNITE-5789 merge [1] introduces this bug: > > > > Additional cache is being started from a template (cache name ends with > *). > > Normally template caches are only started when a cache with matching name > > has been requested. > > > > Pavel > > > > [1] > > > > https://github.com/apache/ignite/commit/d821d0999749a1be318a2106d73654 > 2272a42ab0 > > > > On Fri, May 25, 2018 at 8:23 PM, Dmitriy Setrakyan <d...@gridgain.com> > wrote: > > > > > Hi Pavel, can you please respond here? > > > > > > ---------- Forwarded message ---------- > > > From: Dmitry Pavlov <dpavlov....@gmail.com> > > > Date: Fri, May 25, 2018 at 5:08 AM > > > Subject: Platform .NET add to RunAll Basic suite > > > To: dev <dev@ignite.apache.org> > > > > > > > > > Hi Igniters, > > > > > > recently we've got 60-70 new test failures in .NET > > > > > > For me it is not simple to say which commit has failed build, so I > > suggest > > > the following: > > > > > > 1. Add Platform .NET tests to run-all basic, it will be started by > > > per-VCS-commit basis. Per commit run will indicate change which failed > > the > > > build. > > > > > > 2. Find out and mute flaky tests with tickets (if any). > > > > > > WDYT? > > > > > > Sincerely, > > > Dmitriy Pavlov > > > > > > > > >
