Ignite 2.7.5 Release scope

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

Re: Ignite 2.7.5 Release scope

Ivan Pavlukhin
Dmitry,

It seems that a lunch becomes more and more expensive =(

Just a thought regarding a particular error, we can borrow some ideas
from netty [1].

[1] https://github.com/netty/netty/blob/b2eaab092b68f3c61f1e14f39e38c964e5095dd7/common/src/main/java/io/netty/util/internal/PlatformDependent0.java#L469

ср, 27 мар. 2019 г. в 20:33, Dmitriy Pavlov <[hidden email]>:

>
> I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> start without changes:
>
> Exception in thread "main" java.lang.ExceptionInInitializerError
> at
> org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> at org.apache.ignite.Ignition.start(Ignition.java:348)
> at
> org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> Method)
> at
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> class is unavailable.
> at
> org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> ... 9 more
> Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> at
> java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> at
> java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> at java.base/java.lang.Class.forName0(Native Method)
> at java.base/java.lang.Class.forName(Class.java:332)
> at
> org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> ... 10 more
>
> I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> correct because for 12 we need some changes.
>
>
> ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
>
> > Hello!
> >
> > So after rebasing I was actually able to get it to pass:
> >
> > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > - Java 11, passed
> >
> > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > - Java 11 without fix, will fail every time
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> >
> > > Hello.
> > >
> > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > commit is positioned on top of especially unlucky HEAD.
> > >
> > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > other
> > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > >
> > > It will very reliably hang on Windows/Java 11 without patch and will
> > > always pass with my patch (and TLSv1.2).
> > >
> > > So no additional test is needed - we are testing a known regression here.
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > [hidden email]
> > > >:
> > >
> > >> Hello Ilya,
> > >>
> > >> I do not see any issues with the mentioned test. I see the following
> > >> output
> > >> in the logs:
> > >>
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > >> Stopping test:
> > >>
> > >>
> > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > >> in 37768 ms <<<
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >> Starting test class: IgniteClientConnectTest <<<
> > >>
> > >> The issue with Windows may be long connection timeouts, in this case we
> > >> should either split the suite into multiple ones or decrease the SPI
> > >> timeouts.
> > >>
> > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > >:
> > >>
> > >> > Hello!
> > >> >
> > >> > It seems that I can no longer test this case, on account of
> > >> >
> > >> >
> > >>
> > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > >> > hanging every time under Java 11 on Windows.
> > >> >
> > >> > Alexey, Ivan, can you please take a look?
> > >> >
> > >> >
> > >> >
> > >>
> > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> >
> > >> > Regards,
> > >> >
> > >> > --
> > >> > Ilya Kasnacheev
> > >> >
> > >> >
> > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > [hidden email]
> > >> >:
> > >> >
> > >> > > Hello!
> > >> > >
> > >> > > Basically there is a test that explicitly highlights this problem,
> > >> that
> > >> > is
> > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > >> pass
> > >> > > with this patch.
> > >> > >
> > >> > > I have started that on TC, results will probably be available later
> > >> > today:
> > >> > >
> > >> > >
> > >> >
> > >>
> > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > > (mind the Java version).
> > >> > >
> > >> > > Regards,
> > >> > > --
> > >> > > Ilya Kasnacheev
> > >> > >
> > >> > >
> > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > >> > >
> > >> > >> Dmitry, Ilya,
> > >> > >>
> > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > >> local
> > >> > >> PR.  Basically, changes look good to me.
> > >> > >>
> > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > >> > >> details and I haven't tested these changes under Java 11. One more
> > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > >> > >> explicitly highlight the problem being solved.
> > >> > >>
> > >> > >>
> > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > >>
> > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > >> > wrote:
> > >> > >> >
> > >> > >> > Hi Igniters,
> > >> > >> >
> > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > busy
> > >> > wait
> > >> > >> on
> > >> > >> > processWrite during SSL handshake.
> > >> > >> > seems to be blocker cause it is related to Java 11
> > >> > >> >
> > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > >> comments
> > >> > >> > addressed?
> > >> > >> >
> > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > located
> > >> in
> > >> > >> > GridNioServer.
> > >> > >> >
> > >> > >> > Sincerely,
> > >> > >> > Dmitriy Pavlov
> > >> > >> >
> > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > >> > >> >
> > >> > >> >
> > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > >> > >> >
> > >> > >> > > Huge +1
> > >> > >> > >
> > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > >> > >> > >
> > >> > >> > >
> > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > >
> > >> > >> wrote:
> > >> > >> > > >
> > >> > >> > > > Hi Igniters,
> > >> > >> > > >
> > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > our
> > >> > new
> > >> > >> tests
> > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > >> > >> > > >
> > >> > >> > > > Sincerely,
> > >> > >> > > > Dmitriy Pavlov
> > >> > >> > > >
> > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > >> [hidden email]>:
> > >> > >> > > >
> > >> > >> > > >> Hi Ignite Developers,
> > >> > >> > > >>
> > >> > >> > > >> In a separate discussion, I've shared a log with all
> > commits.
> > >> > >> > > >>
> > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > >> so
> > >> > the
> > >> > >> > > scope
> > >> > >> > > >> of release will be discussed in another way: only explicitly
> > >> > >> declared
> > >> > >> > > >> commits will be cherry-picked.
> > >> > >> > > >>
> > >> > >> > > >> Sincerely,
> > >> > >> > > >> Dmitriy Pavlov
> > >> > >> > > >>
> > >> > >> > >
> > >> > >> > >
> > >> > >>
> > >> > >
> > >> >
> > >>
> > >
> >



--
Best regards,
Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

agura
In reply to this post by Dmitry Pavlov
It seems that SharedSecrets is moved to jdk.internal.access package.
See PR for OpenJ9 [1].

I believe we should just add export of jdk.internal.access package to
our scripts and TC configurations.

[1] https://github.com/eclipse/openj9/pull/3824

On Wed, Mar 27, 2019 at 8:33 PM Dmitriy Pavlov <[hidden email]> wrote:

>
> I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> start without changes:
>
> Exception in thread "main" java.lang.ExceptionInInitializerError
> at
> org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> at org.apache.ignite.Ignition.start(Ignition.java:348)
> at
> org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> Method)
> at
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> class is unavailable.
> at
> org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> ... 9 more
> Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> at
> java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> at
> java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> at java.base/java.lang.Class.forName0(Native Method)
> at java.base/java.lang.Class.forName(Class.java:332)
> at
> org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> ... 10 more
>
> I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> correct because for 12 we need some changes.
>
>
> ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
>
> > Hello!
> >
> > So after rebasing I was actually able to get it to pass:
> >
> > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > - Java 11, passed
> >
> > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > - Java 11 without fix, will fail every time
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> >
> > > Hello.
> > >
> > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > commit is positioned on top of especially unlucky HEAD.
> > >
> > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > other
> > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > >
> > > It will very reliably hang on Windows/Java 11 without patch and will
> > > always pass with my patch (and TLSv1.2).
> > >
> > > So no additional test is needed - we are testing a known regression here.
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > [hidden email]
> > > >:
> > >
> > >> Hello Ilya,
> > >>
> > >> I do not see any issues with the mentioned test. I see the following
> > >> output
> > >> in the logs:
> > >>
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > >> Stopping test:
> > >>
> > >>
> > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > >> in 37768 ms <<<
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >> Starting test class: IgniteClientConnectTest <<<
> > >>
> > >> The issue with Windows may be long connection timeouts, in this case we
> > >> should either split the suite into multiple ones or decrease the SPI
> > >> timeouts.
> > >>
> > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > >:
> > >>
> > >> > Hello!
> > >> >
> > >> > It seems that I can no longer test this case, on account of
> > >> >
> > >> >
> > >>
> > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > >> > hanging every time under Java 11 on Windows.
> > >> >
> > >> > Alexey, Ivan, can you please take a look?
> > >> >
> > >> >
> > >> >
> > >>
> > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> >
> > >> > Regards,
> > >> >
> > >> > --
> > >> > Ilya Kasnacheev
> > >> >
> > >> >
> > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > [hidden email]
> > >> >:
> > >> >
> > >> > > Hello!
> > >> > >
> > >> > > Basically there is a test that explicitly highlights this problem,
> > >> that
> > >> > is
> > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > >> pass
> > >> > > with this patch.
> > >> > >
> > >> > > I have started that on TC, results will probably be available later
> > >> > today:
> > >> > >
> > >> > >
> > >> >
> > >>
> > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > > (mind the Java version).
> > >> > >
> > >> > > Regards,
> > >> > > --
> > >> > > Ilya Kasnacheev
> > >> > >
> > >> > >
> > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > >> > >
> > >> > >> Dmitry, Ilya,
> > >> > >>
> > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > >> local
> > >> > >> PR.  Basically, changes look good to me.
> > >> > >>
> > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > >> > >> details and I haven't tested these changes under Java 11. One more
> > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > >> > >> explicitly highlight the problem being solved.
> > >> > >>
> > >> > >>
> > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > >>
> > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > >> > wrote:
> > >> > >> >
> > >> > >> > Hi Igniters,
> > >> > >> >
> > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > busy
> > >> > wait
> > >> > >> on
> > >> > >> > processWrite during SSL handshake.
> > >> > >> > seems to be blocker cause it is related to Java 11
> > >> > >> >
> > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > >> comments
> > >> > >> > addressed?
> > >> > >> >
> > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > located
> > >> in
> > >> > >> > GridNioServer.
> > >> > >> >
> > >> > >> > Sincerely,
> > >> > >> > Dmitriy Pavlov
> > >> > >> >
> > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > >> > >> >
> > >> > >> >
> > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > >> > >> >
> > >> > >> > > Huge +1
> > >> > >> > >
> > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > >> > >> > >
> > >> > >> > >
> > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > >
> > >> > >> wrote:
> > >> > >> > > >
> > >> > >> > > > Hi Igniters,
> > >> > >> > > >
> > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > our
> > >> > new
> > >> > >> tests
> > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > >> > >> > > >
> > >> > >> > > > Sincerely,
> > >> > >> > > > Dmitriy Pavlov
> > >> > >> > > >
> > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > >> [hidden email]>:
> > >> > >> > > >
> > >> > >> > > >> Hi Ignite Developers,
> > >> > >> > > >>
> > >> > >> > > >> In a separate discussion, I've shared a log with all
> > commits.
> > >> > >> > > >>
> > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > >> so
> > >> > the
> > >> > >> > > scope
> > >> > >> > > >> of release will be discussed in another way: only explicitly
> > >> > >> declared
> > >> > >> > > >> commits will be cherry-picked.
> > >> > >> > > >>
> > >> > >> > > >> Sincerely,
> > >> > >> > > >> Dmitriy Pavlov
> > >> > >> > > >>
> > >> > >> > >
> > >> > >> > >
> > >> > >>
> > >> > >
> > >> >
> > >>
> > >
> >
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

agura
In reply to this post by Ivan Pavlukhin
>> Just a thought regarding a particular error, we can borrow some ideas
from netty.

We are going in the similar way:
https://github.com/apache/ignite/blob/master/modules/core/src/main/java/org/apache/ignite/internal/util/GridUnsafe.java#L1467

On Wed, Mar 27, 2019 at 9:38 PM Павлухин Иван <[hidden email]> wrote:

>
> Dmitry,
>
> It seems that a lunch becomes more and more expensive =(
>
> Just a thought regarding a particular error, we can borrow some ideas
> from netty [1].
>
> [1] https://github.com/netty/netty/blob/b2eaab092b68f3c61f1e14f39e38c964e5095dd7/common/src/main/java/io/netty/util/internal/PlatformDependent0.java#L469
>
> ср, 27 мар. 2019 г. в 20:33, Dmitriy Pavlov <[hidden email]>:
> >
> > I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> > start without changes:
> >
> > Exception in thread "main" java.lang.ExceptionInInitializerError
> > at
> > org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> > at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> > at org.apache.ignite.Ignition.start(Ignition.java:348)
> > at
> > org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> > at
> > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> > at
> > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> > at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> > at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> > Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> > class is unavailable.
> > at
> > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> > at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> > ... 9 more
> > Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> > at
> > java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> > at
> > java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> > at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> > at java.base/java.lang.Class.forName0(Native Method)
> > at java.base/java.lang.Class.forName(Class.java:332)
> > at
> > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> > ... 10 more
> >
> > I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> > correct because for 12 we need some changes.
> >
> >
> > ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
> >
> > > Hello!
> > >
> > > So after rebasing I was actually able to get it to pass:
> > >
> > > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > > - Java 11, passed
> > >
> > > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > > - Java 11 without fix, will fail every time
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> > >
> > > > Hello.
> > > >
> > > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > > commit is positioned on top of especially unlucky HEAD.
> > > >
> > > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > > other
> > > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > > >
> > > > It will very reliably hang on Windows/Java 11 without patch and will
> > > > always pass with my patch (and TLSv1.2).
> > > >
> > > > So no additional test is needed - we are testing a known regression here.
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > > [hidden email]
> > > > >:
> > > >
> > > >> Hello Ilya,
> > > >>
> > > >> I do not see any issues with the mentioned test. I see the following
> > > >> output
> > > >> in the logs:
> > > >>
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > > >> Stopping test:
> > > >>
> > > >>
> > > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > >> in 37768 ms <<<
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > >> Starting test class: IgniteClientConnectTest <<<
> > > >>
> > > >> The issue with Windows may be long connection timeouts, in this case we
> > > >> should either split the suite into multiple ones or decrease the SPI
> > > >> timeouts.
> > > >>
> > > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > > >:
> > > >>
> > > >> > Hello!
> > > >> >
> > > >> > It seems that I can no longer test this case, on account of
> > > >> >
> > > >> >
> > > >>
> > > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > >> > hanging every time under Java 11 on Windows.
> > > >> >
> > > >> > Alexey, Ivan, can you please take a look?
> > > >> >
> > > >> >
> > > >> >
> > > >>
> > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >> >
> > > >> > Regards,
> > > >> >
> > > >> > --
> > > >> > Ilya Kasnacheev
> > > >> >
> > > >> >
> > > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > [hidden email]
> > > >> >:
> > > >> >
> > > >> > > Hello!
> > > >> > >
> > > >> > > Basically there is a test that explicitly highlights this problem,
> > > >> that
> > > >> > is
> > > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > > >> pass
> > > >> > > with this patch.
> > > >> > >
> > > >> > > I have started that on TC, results will probably be available later
> > > >> > today:
> > > >> > >
> > > >> > >
> > > >> >
> > > >>
> > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >> > > (mind the Java version).
> > > >> > >
> > > >> > > Regards,
> > > >> > > --
> > > >> > > Ilya Kasnacheev
> > > >> > >
> > > >> > >
> > > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > > >> > >
> > > >> > >> Dmitry, Ilya,
> > > >> > >>
> > > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > > >> local
> > > >> > >> PR.  Basically, changes look good to me.
> > > >> > >>
> > > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > > >> > >> details and I haven't tested these changes under Java 11. One more
> > > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > > >> > >> explicitly highlight the problem being solved.
> > > >> > >>
> > > >> > >>
> > > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > > >> > >>
> > > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > > >> > wrote:
> > > >> > >> >
> > > >> > >> > Hi Igniters,
> > > >> > >> >
> > > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > > busy
> > > >> > wait
> > > >> > >> on
> > > >> > >> > processWrite during SSL handshake.
> > > >> > >> > seems to be blocker cause it is related to Java 11
> > > >> > >> >
> > > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > >> comments
> > > >> > >> > addressed?
> > > >> > >> >
> > > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > located
> > > >> in
> > > >> > >> > GridNioServer.
> > > >> > >> >
> > > >> > >> > Sincerely,
> > > >> > >> > Dmitriy Pavlov
> > > >> > >> >
> > > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > > >> > >> >
> > > >> > >> >
> > > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > > >> > >> >
> > > >> > >> > > Huge +1
> > > >> > >> > >
> > > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > >> > >> > >
> > > >> > >> > >
> > > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > > >
> > > >> > >> wrote:
> > > >> > >> > > >
> > > >> > >> > > > Hi Igniters,
> > > >> > >> > > >
> > > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > > our
> > > >> > new
> > > >> > >> tests
> > > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > > >> > >> > > >
> > > >> > >> > > > Sincerely,
> > > >> > >> > > > Dmitriy Pavlov
> > > >> > >> > > >
> > > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > > >> [hidden email]>:
> > > >> > >> > > >
> > > >> > >> > > >> Hi Ignite Developers,
> > > >> > >> > > >>
> > > >> > >> > > >> In a separate discussion, I've shared a log with all
> > > commits.
> > > >> > >> > > >>
> > > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > > >> so
> > > >> > the
> > > >> > >> > > scope
> > > >> > >> > > >> of release will be discussed in another way: only explicitly
> > > >> > >> declared
> > > >> > >> > > >> commits will be cherry-picked.
> > > >> > >> > > >>
> > > >> > >> > > >> Sincerely,
> > > >> > >> > > >> Dmitriy Pavlov
> > > >> > >> > > >>
> > > >> > >> > >
> > > >> > >> > >
> > > >> > >>
> > > >> > >
> > > >> >
> > > >>
> > > >
> > >
>
>
>
> --
> Best regards,
> Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

agura
In reply to this post by agura
>>> I believe we should just add export of jdk.internal.access package to
our scripts and TC configurations.

Unfortunately it isn't enough! We have to modify
GridUnsafe#javaNioAccessObject method.

On Wed, Mar 27, 2019 at 10:08 PM Andrey Gura <[hidden email]> wrote:

>
> It seems that SharedSecrets is moved to jdk.internal.access package.
> See PR for OpenJ9 [1].
>
> I believe we should just add export of jdk.internal.access package to
> our scripts and TC configurations.
>
> [1] https://github.com/eclipse/openj9/pull/3824
>
> On Wed, Mar 27, 2019 at 8:33 PM Dmitriy Pavlov <[hidden email]> wrote:
> >
> > I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> > start without changes:
> >
> > Exception in thread "main" java.lang.ExceptionInInitializerError
> > at
> > org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> > at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> > at org.apache.ignite.Ignition.start(Ignition.java:348)
> > at
> > org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> > Method)
> > at
> > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> > at
> > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> > at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> > at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> > Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> > class is unavailable.
> > at
> > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> > at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> > ... 9 more
> > Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> > at
> > java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> > at
> > java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> > at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> > at java.base/java.lang.Class.forName0(Native Method)
> > at java.base/java.lang.Class.forName(Class.java:332)
> > at
> > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> > ... 10 more
> >
> > I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> > correct because for 12 we need some changes.
> >
> >
> > ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
> >
> > > Hello!
> > >
> > > So after rebasing I was actually able to get it to pass:
> > >
> > > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > > - Java 11, passed
> > >
> > > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > > - Java 11 without fix, will fail every time
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> > >
> > > > Hello.
> > > >
> > > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > > commit is positioned on top of especially unlucky HEAD.
> > > >
> > > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > > other
> > > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > > >
> > > > It will very reliably hang on Windows/Java 11 without patch and will
> > > > always pass with my patch (and TLSv1.2).
> > > >
> > > > So no additional test is needed - we are testing a known regression here.
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > > [hidden email]
> > > > >:
> > > >
> > > >> Hello Ilya,
> > > >>
> > > >> I do not see any issues with the mentioned test. I see the following
> > > >> output
> > > >> in the logs:
> > > >>
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > > >> Stopping test:
> > > >>
> > > >>
> > > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > >> in 37768 ms <<<
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > >> Starting test class: IgniteClientConnectTest <<<
> > > >>
> > > >> The issue with Windows may be long connection timeouts, in this case we
> > > >> should either split the suite into multiple ones or decrease the SPI
> > > >> timeouts.
> > > >>
> > > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > > >:
> > > >>
> > > >> > Hello!
> > > >> >
> > > >> > It seems that I can no longer test this case, on account of
> > > >> >
> > > >> >
> > > >>
> > > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > >> > hanging every time under Java 11 on Windows.
> > > >> >
> > > >> > Alexey, Ivan, can you please take a look?
> > > >> >
> > > >> >
> > > >> >
> > > >>
> > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >> >
> > > >> > Regards,
> > > >> >
> > > >> > --
> > > >> > Ilya Kasnacheev
> > > >> >
> > > >> >
> > > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > [hidden email]
> > > >> >:
> > > >> >
> > > >> > > Hello!
> > > >> > >
> > > >> > > Basically there is a test that explicitly highlights this problem,
> > > >> that
> > > >> > is
> > > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > > >> pass
> > > >> > > with this patch.
> > > >> > >
> > > >> > > I have started that on TC, results will probably be available later
> > > >> > today:
> > > >> > >
> > > >> > >
> > > >> >
> > > >>
> > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >> > > (mind the Java version).
> > > >> > >
> > > >> > > Regards,
> > > >> > > --
> > > >> > > Ilya Kasnacheev
> > > >> > >
> > > >> > >
> > > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > > >> > >
> > > >> > >> Dmitry, Ilya,
> > > >> > >>
> > > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > > >> local
> > > >> > >> PR.  Basically, changes look good to me.
> > > >> > >>
> > > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > > >> > >> details and I haven't tested these changes under Java 11. One more
> > > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > > >> > >> explicitly highlight the problem being solved.
> > > >> > >>
> > > >> > >>
> > > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > > >> > >>
> > > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > > >> > wrote:
> > > >> > >> >
> > > >> > >> > Hi Igniters,
> > > >> > >> >
> > > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > > busy
> > > >> > wait
> > > >> > >> on
> > > >> > >> > processWrite during SSL handshake.
> > > >> > >> > seems to be blocker cause it is related to Java 11
> > > >> > >> >
> > > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > >> comments
> > > >> > >> > addressed?
> > > >> > >> >
> > > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > located
> > > >> in
> > > >> > >> > GridNioServer.
> > > >> > >> >
> > > >> > >> > Sincerely,
> > > >> > >> > Dmitriy Pavlov
> > > >> > >> >
> > > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > > >> > >> >
> > > >> > >> >
> > > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > > >> > >> >
> > > >> > >> > > Huge +1
> > > >> > >> > >
> > > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > >> > >> > >
> > > >> > >> > >
> > > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > > >
> > > >> > >> wrote:
> > > >> > >> > > >
> > > >> > >> > > > Hi Igniters,
> > > >> > >> > > >
> > > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > > our
> > > >> > new
> > > >> > >> tests
> > > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > > >> > >> > > >
> > > >> > >> > > > Sincerely,
> > > >> > >> > > > Dmitriy Pavlov
> > > >> > >> > > >
> > > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > > >> [hidden email]>:
> > > >> > >> > > >
> > > >> > >> > > >> Hi Ignite Developers,
> > > >> > >> > > >>
> > > >> > >> > > >> In a separate discussion, I've shared a log with all
> > > commits.
> > > >> > >> > > >>
> > > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > > >> so
> > > >> > the
> > > >> > >> > > scope
> > > >> > >> > > >> of release will be discussed in another way: only explicitly
> > > >> > >> declared
> > > >> > >> > > >> commits will be cherry-picked.
> > > >> > >> > > >>
> > > >> > >> > > >> Sincerely,
> > > >> > >> > > >> Dmitriy Pavlov
> > > >> > >> > > >>
> > > >> > >> > >
> > > >> > >> > >
> > > >> > >>
> > > >> > >
> > > >> >
> > > >>
> > > >
> > >
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Ivan Pavlukhin
Andrey,

There is no method JavaNioAccess.newDirectByteBuffer which is used in
our code caused the problem.

> We are going in the similar way:

Our way is similar, but it does not work in java 12.

I think it is for separate thread. Let's stop live coding =)

ср, 27 мар. 2019 г. в 22:15, Andrey Gura <[hidden email]>:

>
> >>> I believe we should just add export of jdk.internal.access package to
> our scripts and TC configurations.
>
> Unfortunately it isn't enough! We have to modify
> GridUnsafe#javaNioAccessObject method.
>
> On Wed, Mar 27, 2019 at 10:08 PM Andrey Gura <[hidden email]> wrote:
> >
> > It seems that SharedSecrets is moved to jdk.internal.access package.
> > See PR for OpenJ9 [1].
> >
> > I believe we should just add export of jdk.internal.access package to
> > our scripts and TC configurations.
> >
> > [1] https://github.com/eclipse/openj9/pull/3824
> >
> > On Wed, Mar 27, 2019 at 8:33 PM Dmitriy Pavlov <[hidden email]> wrote:
> > >
> > > I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> > > start without changes:
> > >
> > > Exception in thread "main" java.lang.ExceptionInInitializerError
> > > at
> > > org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> > > at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> > > at org.apache.ignite.Ignition.start(Ignition.java:348)
> > > at
> > > org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> > > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> > > Method)
> > > at
> > > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> > > at
> > > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> > > at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> > > at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> > > Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> > > class is unavailable.
> > > at
> > > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> > > at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> > > ... 9 more
> > > Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> > > at
> > > java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> > > at
> > > java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> > > at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> > > at java.base/java.lang.Class.forName0(Native Method)
> > > at java.base/java.lang.Class.forName(Class.java:332)
> > > at
> > > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> > > ... 10 more
> > >
> > > I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> > > correct because for 12 we need some changes.
> > >
> > >
> > > ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
> > >
> > > > Hello!
> > > >
> > > > So after rebasing I was actually able to get it to pass:
> > > >
> > > > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > > > - Java 11, passed
> > > >
> > > > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > > > - Java 11 without fix, will fail every time
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> > > >
> > > > > Hello.
> > > > >
> > > > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > > > commit is positioned on top of especially unlucky HEAD.
> > > > >
> > > > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > > > other
> > > > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > > > >
> > > > > It will very reliably hang on Windows/Java 11 without patch and will
> > > > > always pass with my patch (and TLSv1.2).
> > > > >
> > > > > So no additional test is needed - we are testing a known regression here.
> > > > >
> > > > > Regards,
> > > > > --
> > > > > Ilya Kasnacheev
> > > > >
> > > > >
> > > > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > > > [hidden email]
> > > > > >:
> > > > >
> > > > >> Hello Ilya,
> > > > >>
> > > > >> I do not see any issues with the mentioned test. I see the following
> > > > >> output
> > > > >> in the logs:
> > > > >>
> > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > > > >> Stopping test:
> > > > >>
> > > > >>
> > > > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > > >> in 37768 ms <<<
> > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > > >> Starting test class: IgniteClientConnectTest <<<
> > > > >>
> > > > >> The issue with Windows may be long connection timeouts, in this case we
> > > > >> should either split the suite into multiple ones or decrease the SPI
> > > > >> timeouts.
> > > > >>
> > > > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > > > >:
> > > > >>
> > > > >> > Hello!
> > > > >> >
> > > > >> > It seems that I can no longer test this case, on account of
> > > > >> >
> > > > >> >
> > > > >>
> > > > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > > >> > hanging every time under Java 11 on Windows.
> > > > >> >
> > > > >> > Alexey, Ivan, can you please take a look?
> > > > >> >
> > > > >> >
> > > > >> >
> > > > >>
> > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > >> >
> > > > >> > Regards,
> > > > >> >
> > > > >> > --
> > > > >> > Ilya Kasnacheev
> > > > >> >
> > > > >> >
> > > > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > > [hidden email]
> > > > >> >:
> > > > >> >
> > > > >> > > Hello!
> > > > >> > >
> > > > >> > > Basically there is a test that explicitly highlights this problem,
> > > > >> that
> > > > >> > is
> > > > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > > > >> pass
> > > > >> > > with this patch.
> > > > >> > >
> > > > >> > > I have started that on TC, results will probably be available later
> > > > >> > today:
> > > > >> > >
> > > > >> > >
> > > > >> >
> > > > >>
> > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > >> > > (mind the Java version).
> > > > >> > >
> > > > >> > > Regards,
> > > > >> > > --
> > > > >> > > Ilya Kasnacheev
> > > > >> > >
> > > > >> > >
> > > > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > > > >> > >
> > > > >> > >> Dmitry, Ilya,
> > > > >> > >>
> > > > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > > > >> local
> > > > >> > >> PR.  Basically, changes look good to me.
> > > > >> > >>
> > > > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > > > >> > >> details and I haven't tested these changes under Java 11. One more
> > > > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > > > >> > >> explicitly highlight the problem being solved.
> > > > >> > >>
> > > > >> > >>
> > > > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > > > >> > >>
> > > > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > > > >> > wrote:
> > > > >> > >> >
> > > > >> > >> > Hi Igniters,
> > > > >> > >> >
> > > > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > > > busy
> > > > >> > wait
> > > > >> > >> on
> > > > >> > >> > processWrite during SSL handshake.
> > > > >> > >> > seems to be blocker cause it is related to Java 11
> > > > >> > >> >
> > > > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > > >> comments
> > > > >> > >> > addressed?
> > > > >> > >> >
> > > > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > > located
> > > > >> in
> > > > >> > >> > GridNioServer.
> > > > >> > >> >
> > > > >> > >> > Sincerely,
> > > > >> > >> > Dmitriy Pavlov
> > > > >> > >> >
> > > > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > > > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > > > >> > >> >
> > > > >> > >> >
> > > > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > > > >> > >> >
> > > > >> > >> > > Huge +1
> > > > >> > >> > >
> > > > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > > >> > >> > >
> > > > >> > >> > >
> > > > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > > > >
> > > > >> > >> wrote:
> > > > >> > >> > > >
> > > > >> > >> > > > Hi Igniters,
> > > > >> > >> > > >
> > > > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > > > our
> > > > >> > new
> > > > >> > >> tests
> > > > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > > > >> > >> > > >
> > > > >> > >> > > > Sincerely,
> > > > >> > >> > > > Dmitriy Pavlov
> > > > >> > >> > > >
> > > > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > > > >> [hidden email]>:
> > > > >> > >> > > >
> > > > >> > >> > > >> Hi Ignite Developers,
> > > > >> > >> > > >>
> > > > >> > >> > > >> In a separate discussion, I've shared a log with all
> > > > commits.
> > > > >> > >> > > >>
> > > > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > > > >> so
> > > > >> > the
> > > > >> > >> > > scope
> > > > >> > >> > > >> of release will be discussed in another way: only explicitly
> > > > >> > >> declared
> > > > >> > >> > > >> commits will be cherry-picked.
> > > > >> > >> > > >>
> > > > >> > >> > > >> Sincerely,
> > > > >> > >> > > >> Dmitriy Pavlov
> > > > >> > >> > > >>
> > > > >> > >> > >
> > > > >> > >> > >
> > > > >> > >>
> > > > >> > >
> > > > >> >
> > > > >>
> > > > >
> > > >



--
Best regards,
Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

agura
Ivan,

I've already point to problem place. So we just have to fix it.

On Wed, Mar 27, 2019 at 10:16 PM Павлухин Иван <[hidden email]> wrote:

>
> Andrey,
>
> There is no method JavaNioAccess.newDirectByteBuffer which is used in
> our code caused the problem.
>
> > We are going in the similar way:
>
> Our way is similar, but it does not work in java 12.
>
> I think it is for separate thread. Let's stop live coding =)
>
> ср, 27 мар. 2019 г. в 22:15, Andrey Gura <[hidden email]>:
> >
> > >>> I believe we should just add export of jdk.internal.access package to
> > our scripts and TC configurations.
> >
> > Unfortunately it isn't enough! We have to modify
> > GridUnsafe#javaNioAccessObject method.
> >
> > On Wed, Mar 27, 2019 at 10:08 PM Andrey Gura <[hidden email]> wrote:
> > >
> > > It seems that SharedSecrets is moved to jdk.internal.access package.
> > > See PR for OpenJ9 [1].
> > >
> > > I believe we should just add export of jdk.internal.access package to
> > > our scripts and TC configurations.
> > >
> > > [1] https://github.com/eclipse/openj9/pull/3824
> > >
> > > On Wed, Mar 27, 2019 at 8:33 PM Dmitriy Pavlov <[hidden email]> wrote:
> > > >
> > > > I've played with 2.7.5-RC-1 and Java 12, and unfortunately, Ignite can't
> > > > start without changes:
> > > >
> > > > Exception in thread "main" java.lang.ExceptionInInitializerError
> > > > at
> > > > org.apache.ignite.internal.util.IgniteUtils.<clinit>(IgniteUtils.java:795)
> > > > at org.apache.ignite.internal.IgnitionEx.<clinit>(IgnitionEx.java:209)
> > > > at org.apache.ignite.Ignition.start(Ignition.java:348)
> > > > at
> > > > org.apache.ignite.examples.ExampleNodeStartup.main(ExampleNodeStartup.java:34)
> > > > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
> > > > Method)
> > > > at
> > > > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> > > > at
> > > > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> > > > at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> > > > at com.intellij.rt.execution.application.AppMainV2.main(AppMainV2.java:131)
> > > > Caused by: java.lang.RuntimeException: jdk.internal.misc.JavaNioAccess
> > > > class is unavailable.
> > > > at
> > > > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1453)
> > > > at org.apache.ignite.internal.util.GridUnsafe.<clinit>(GridUnsafe.java:112)
> > > > ... 9 more
> > > > Caused by: java.lang.ClassNotFoundException: jdk.internal.misc.SharedSecrets
> > > > at
> > > > java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
> > > > at
> > > > java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
> > > > at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
> > > > at java.base/java.lang.Class.forName0(Native Method)
> > > > at java.base/java.lang.Class.forName(Class.java:332)
> > > > at
> > > > org.apache.ignite.internal.util.GridUnsafe.javaNioAccessObject(GridUnsafe.java:1446)
> > > > ... 10 more
> > > >
> > > > I'm not sure we can workaround it. So probably fix with JDK.ver>11 is not
> > > > correct because for 12 we need some changes.
> > > >
> > > >
> > > > ср, 27 мар. 2019 г. в 17:44, Ilya Kasnacheev <[hidden email]>:
> > > >
> > > > > Hello!
> > > > >
> > > > > So after rebasing I was actually able to get it to pass:
> > > > >
> > > > > https://ci.ignite.apache.org/viewLog.html?buildId=3431647&buildTypeId=IgniteTests24Java8_SpiWindows
> > > > > - Java 11, passed
> > > > >
> > > > > https://ci.ignite.apache.org/viewLog.html?buildId=3431619&buildTypeId=IgniteTests24Java8_SpiWindows
> > > > > - Java 11 without fix, will fail every time
> > > > >
> > > > > Regards,
> > > > > --
> > > > > Ilya Kasnacheev
> > > > >
> > > > >
> > > > > вт, 26 мар. 2019 г. в 19:41, Ilya Kasnacheev <[hidden email]>:
> > > > >
> > > > > > Hello.
> > > > > >
> > > > > > Yes, locally this test seems to pass. However, no luck on TC. Maybe my
> > > > > > commit is positioned on top of especially unlucky HEAD.
> > > > > >
> > > > > > Anyway, my point was thatTcpDiscoverySslTrustedUntrustedTest (or any
> > > > > other
> > > > > > intra-node SSL test) is a sufficient test for IGNITE-11299.
> > > > > >
> > > > > > It will very reliably hang on Windows/Java 11 without patch and will
> > > > > > always pass with my patch (and TLSv1.2).
> > > > > >
> > > > > > So no additional test is needed - we are testing a known regression here.
> > > > > >
> > > > > > Regards,
> > > > > > --
> > > > > > Ilya Kasnacheev
> > > > > >
> > > > > >
> > > > > > вт, 26 мар. 2019 г. в 13:43, Alexey Goncharuk <
> > > > > [hidden email]
> > > > > > >:
> > > > > >
> > > > > >> Hello Ilya,
> > > > > >>
> > > > > >> I do not see any issues with the mentioned test. I see the following
> > > > > >> output
> > > > > >> in the logs:
> > > > > >>
> > > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > > > > >> Stopping test:
> > > > > >>
> > > > > >>
> > > > > TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > > > >> in 37768 ms <<<
> > > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > > > >> Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > > > >> [21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > > > > >> Starting test class: IgniteClientConnectTest <<<
> > > > > >>
> > > > > >> The issue with Windows may be long connection timeouts, in this case we
> > > > > >> should either split the suite into multiple ones or decrease the SPI
> > > > > >> timeouts.
> > > > > >>
> > > > > >> пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <[hidden email]
> > > > > >:
> > > > > >>
> > > > > >> > Hello!
> > > > > >> >
> > > > > >> > It seems that I can no longer test this case, on account of
> > > > > >> >
> > > > > >> >
> > > > > >>
> > > > > TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > > > >> > hanging every time under Java 11 on Windows.
> > > > > >> >
> > > > > >> > Alexey, Ivan, can you please take a look?
> > > > > >> >
> > > > > >> >
> > > > > >> >
> > > > > >>
> > > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > >> >
> > > > > >> > Regards,
> > > > > >> >
> > > > > >> > --
> > > > > >> > Ilya Kasnacheev
> > > > > >> >
> > > > > >> >
> > > > > >> > пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > > > [hidden email]
> > > > > >> >:
> > > > > >> >
> > > > > >> > > Hello!
> > > > > >> > >
> > > > > >> > > Basically there is a test that explicitly highlights this problem,
> > > > > >> that
> > > > > >> > is
> > > > > >> > > running SSL tests on Windows + Java 11. They will hang on Master but
> > > > > >> pass
> > > > > >> > > with this patch.
> > > > > >> > >
> > > > > >> > > I have started that on TC, results will probably be available later
> > > > > >> > today:
> > > > > >> > >
> > > > > >> > >
> > > > > >> >
> > > > > >>
> > > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > >> > > (mind the Java version).
> > > > > >> > >
> > > > > >> > > Regards,
> > > > > >> > > --
> > > > > >> > > Ilya Kasnacheev
> > > > > >> > >
> > > > > >> > >
> > > > > >> > > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <[hidden email]>:
> > > > > >> > >
> > > > > >> > >> Dmitry, Ilya,
> > > > > >> > >>
> > > > > >> > >> Yes, I've looked through those changes [1] as they can affect my
> > > > > >> local
> > > > > >> > >> PR.  Basically, changes look good to me.
> > > > > >> > >>
> > > > > >> > >> I'm not an expert with CommunicationSpi component, so can miss some
> > > > > >> > >> details and I haven't tested these changes under Java 11. One more
> > > > > >> > >> thing I'd like to say, I would add additional tests to PR that will
> > > > > >> > >> explicitly highlight the problem being solved.
> > > > > >> > >>
> > > > > >> > >>
> > > > > >> > >> [1] https://issues.apache.org/jira/browse/IGNITE-11299
> > > > > >> > >>
> > > > > >> > >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <[hidden email]>
> > > > > >> > wrote:
> > > > > >> > >> >
> > > > > >> > >> > Hi Igniters,
> > > > > >> > >> >
> > > > > >> > >> > fix https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > > > > busy
> > > > > >> > wait
> > > > > >> > >> on
> > > > > >> > >> > processWrite during SSL handshake.
> > > > > >> > >> > seems to be blocker cause it is related to Java 11
> > > > > >> > >> >
> > > > > >> > >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > > > >> comments
> > > > > >> > >> > addressed?
> > > > > >> > >> >
> > > > > >> > >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > > > located
> > > > > >> in
> > > > > >> > >> > GridNioServer.
> > > > > >> > >> >
> > > > > >> > >> > Sincerely,
> > > > > >> > >> > Dmitriy Pavlov
> > > > > >> > >> >
> > > > > >> > >> > P.S. a quite obvious ticket came to sope, as well:
> > > > > >> > >> > https://issues.apache.org/jira/browse/IGNITE-11600
> > > > > >> > >> >
> > > > > >> > >> >
> > > > > >> > >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <[hidden email]>:
> > > > > >> > >> >
> > > > > >> > >> > > Huge +1
> > > > > >> > >> > >
> > > > > >> > >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > > > >> > >> > >
> > > > > >> > >> > >
> > > > > >> > >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <[hidden email]
> > > > > >
> > > > > >> > >> wrote:
> > > > > >> > >> > > >
> > > > > >> > >> > > > Hi Igniters,
> > > > > >> > >> > > >
> > > > > >> > >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > > > > our
> > > > > >> > new
> > > > > >> > >> tests
> > > > > >> > >> > > > scripts with a couple of Java builds. WDYT?
> > > > > >> > >> > > >
> > > > > >> > >> > > > Sincerely,
> > > > > >> > >> > > > Dmitriy Pavlov
> > > > > >> > >> > > >
> > > > > >> > >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov <
> > > > > >> [hidden email]>:
> > > > > >> > >> > > >
> > > > > >> > >> > > >> Hi Ignite Developers,
> > > > > >> > >> > > >>
> > > > > >> > >> > > >> In a separate discussion, I've shared a log with all
> > > > > commits.
> > > > > >> > >> > > >>
> > > > > >> > >> > > >> As far as I can see, nobody removed commits from this sheet,
> > > > > >> so
> > > > > >> > the
> > > > > >> > >> > > scope
> > > > > >> > >> > > >> of release will be discussed in another way: only explicitly
> > > > > >> > >> declared
> > > > > >> > >> > > >> commits will be cherry-picked.
> > > > > >> > >> > > >>
> > > > > >> > >> > > >> Sincerely,
> > > > > >> > >> > > >> Dmitriy Pavlov
> > > > > >> > >> > > >>
> > > > > >> > >> > >
> > > > > >> > >> > >
> > > > > >> > >>
> > > > > >> > >
> > > > > >> >
> > > > > >>
> > > > > >
> > > > >
>
>
>
> --
> Best regards,
> Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Roman Shtykh
In reply to this post by Dmitry Pavlov
Dmitriy, the probability of having nodes down with these bugs under heavy load conditions is pretty high. Good candidate for cherry-picking.

[1] https://issues.apache.org/jira/browse/IGNITE-9803[2] https://issues.apache.org/jira/browse/IGNITE-11127
-- Roman
 

    On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy Pavlov <[hidden email]> wrote:  
 
 Hi,

I've cherry-picked this commit. It seems it is critical because it also
fixes storage corruption.

Sincerely,
Dmitriy Pavlov

вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky <[hidden email]
>:

> I suppose this ticket [1] : is very useful too.
>
>
> [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> CorruptedTreeException during simultaneous cache put operations ]
>
> >
> >
> >------- Forwarded message -------
> >From: "Alexey Goncharuk" < [hidden email] >
> >To: dev < [hidden email] >
> >Cc:
> >Subject: Re: Ignite 2.7.5 Release scope
> >Date: Tue, 26 Mar 2019 13:42:59 +0300
> >
> >Hello Ilya,
> >
> >I do not see any issues with the mentioned test. I see the following
> output
> >in the logs:
> >
> >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> >Stopping test:
>
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> >in 37768 ms <<<
> >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> >Starting test class: IgniteClientConnectTest <<<
> >
> >The issue with Windows may be long connection timeouts, in this case we
> >should either split the suite into multiple ones or decrease the SPI
> >timeouts.
> >
> >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev < [hidden email]
> >:
> >
> >> Hello!
> >>
> >> It seems that I can no longer test this case, on account of
> >>
> >>
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> >> hanging every time under Java 11 on Windows.
> >>
> >> Alexey, Ivan, can you please take a look?
> >>
> >>
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >>
> >> Regards,
> >>
> >> --
> >> Ilya Kasnacheev
> >>
> >>
> >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> [hidden email] >:
> >>
> >> > Hello!
> >> >
> >> > Basically there is a test that explicitly highlights this problem,
> that
> >> is
> >> > running SSL tests on Windows + Java 11. They will hang on Master but
> >> pass
> >> > with this patch.
> >> >
> >> > I have started that on TC, results will probably be available later
> >> today:
> >> >
> >> >
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >> > (mind the Java version).
> >> >
> >> > Regards,
> >> > --
> >> > Ilya Kasnacheev
> >> >
> >> >
> >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov < [hidden email] >:
> >> >
> >> >> Dmitry, Ilya,
> >> >>
> >> >> Yes, I've looked through those changes [1] as they can affect my
> local
> >> >> PR.  Basically, changes look good to me.
> >> >>
> >> >> I'm not an expert with CommunicationSpi component, so can miss some
> >> >> details and I haven't tested these changes under Java 11. One more
> >> >> thing I'd like to say, I would add additional tests to PR that will
> >> >> explicitly highlight the problem being solved.
> >> >>
> >> >>
> >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> >> >>
> >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov < [hidden email] >
> >> wrote:
> >> >> >
> >> >> > Hi Igniters,
> >> >> >
> >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299 Avoid busy
> >> wait
> >> >> on
> >> >> > processWrite during SSL handshake.
> >> >> > seems to be blocker cause it is related to Java 11
> >> >> >
> >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> >> comments
> >> >> > addressed?
> >> >> >
> >> >> > The ticket is in Patch Available. Reviewer needed. Changes located
> >> in
> >> >> > GridNioServer.
> >> >> >
> >> >> > Sincerely,
> >> >> > Dmitriy Pavlov
> >> >> >
> >> >> > P.S. a quite obvious ticket came to sope, as well:
> >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> >> >> >
> >> >> >
> >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov < [hidden email] >:
> >> >> >
> >> >> > > Huge +1
> >> >> > >
> >> >> > > Will try to add new JDK in nearest time to our Teamcity.
> >> >> > >
> >> >> > >
> >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov < [hidden email]
> >
> >> >> wrote:
> >> >> > > >
> >> >> > > > Hi Igniters,
> >> >> > > >
> >> >> > > > Meanwhile, Java 12 GA is available. I suggest at least test our
> >> new
> >> >> tests
> >> >> > > > scripts with a couple of Java builds. WDYT?
> >> >> > > >
> >> >> > > > Sincerely,
> >> >> > > > Dmitriy Pavlov
> >> >> > > >
> >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> >> < [hidden email] >:
> >> >> > > >
> >> >> > > >> Hi Ignite Developers,
> >> >> > > >>
> >> >> > > >> In a separate discussion, I've shared a log with all commits.
> >> >> > > >>
> >> >> > > >> As far as I can see, nobody removed commits from this sheet,
> so
> >> the
> >> >> > > scope
> >> >> > > >> of release will be discussed in another way: only explicitly
> >> >> declared
> >> >> > > >> commits will be cherry-picked.
> >> >> > > >>
> >> >> > > >> Sincerely,
> >> >> > > >> Dmitriy Pavlov
> >> >> > > >>
> >> >> > >
> >> >> > >
> >> >>
> >> >
>
>
> --
> Zhenya Stanilovsky
>  
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Ilya Kasnacheev
Hello!

How about adding https://issues.apache.org/jira/browse/IGNITE-11366 which
fixes Python's release directory?

I have stumbled on this issue personally.

Regards,
--
Ilya Kasnacheev


чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]>:

> Dmitriy, the probability of having nodes down with these bugs under heavy
> load conditions is pretty high. Good candidate for cherry-picking.
>
> [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> https://issues.apache.org/jira/browse/IGNITE-11127
> -- Roman
>
>
>     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy Pavlov <
> [hidden email]> wrote:
>
>  Hi,
>
> I've cherry-picked this commit. It seems it is critical because it also
> fixes storage corruption.
>
> Sincerely,
> Dmitriy Pavlov
>
> вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky <[hidden email]
> >:
>
> > I suppose this ticket [1] : is very useful too.
> >
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > CorruptedTreeException during simultaneous cache put operations ]
> >
> > >
> > >
> > >------- Forwarded message -------
> > >From: "Alexey Goncharuk" < [hidden email] >
> > >To: dev < [hidden email] >
> > >Cc:
> > >Subject: Re: Ignite 2.7.5 Release scope
> > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > >
> > >Hello Ilya,
> > >
> > >I do not see any issues with the mentioned test. I see the following
> > output
> > >in the logs:
> > >
> > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root] >>>
> > >Stopping test:
> >
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > >in 37768 ms <<<
> > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root] >>>
> > >Starting test class: IgniteClientConnectTest <<<
> > >
> > >The issue with Windows may be long connection timeouts, in this case we
> > >should either split the suite into multiple ones or decrease the SPI
> > >timeouts.
> > >
> > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> [hidden email]
> > >:
> > >
> > >> Hello!
> > >>
> > >> It seems that I can no longer test this case, on account of
> > >>
> > >>
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > >> hanging every time under Java 11 on Windows.
> > >>
> > >> Alexey, Ivan, can you please take a look?
> > >>
> > >>
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >>
> > >> Regards,
> > >>
> > >> --
> > >> Ilya Kasnacheev
> > >>
> > >>
> > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > [hidden email] >:
> > >>
> > >> > Hello!
> > >> >
> > >> > Basically there is a test that explicitly highlights this problem,
> > that
> > >> is
> > >> > running SSL tests on Windows + Java 11. They will hang on Master but
> > >> pass
> > >> > with this patch.
> > >> >
> > >> > I have started that on TC, results will probably be available later
> > >> today:
> > >> >
> > >> >
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > (mind the Java version).
> > >> >
> > >> > Regards,
> > >> > --
> > >> > Ilya Kasnacheev
> > >> >
> > >> >
> > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov < [hidden email]
> >:
> > >> >
> > >> >> Dmitry, Ilya,
> > >> >>
> > >> >> Yes, I've looked through those changes [1] as they can affect my
> > local
> > >> >> PR.  Basically, changes look good to me.
> > >> >>
> > >> >> I'm not an expert with CommunicationSpi component, so can miss some
> > >> >> details and I haven't tested these changes under Java 11. One more
> > >> >> thing I'd like to say, I would add additional tests to PR that will
> > >> >> explicitly highlight the problem being solved.
> > >> >>
> > >> >>
> > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> > >> >>
> > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov < [hidden email]
> >
> > >> wrote:
> > >> >> >
> > >> >> > Hi Igniters,
> > >> >> >
> > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> busy
> > >> wait
> > >> >> on
> > >> >> > processWrite during SSL handshake.
> > >> >> > seems to be blocker cause it is related to Java 11
> > >> >> >
> > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > >> comments
> > >> >> > addressed?
> > >> >> >
> > >> >> > The ticket is in Patch Available. Reviewer needed. Changes
> located
> > >> in
> > >> >> > GridNioServer.
> > >> >> >
> > >> >> > Sincerely,
> > >> >> > Dmitriy Pavlov
> > >> >> >
> > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > >> >> >
> > >> >> >
> > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov < [hidden email]
> >:
> > >> >> >
> > >> >> > > Huge +1
> > >> >> > >
> > >> >> > > Will try to add new JDK in nearest time to our Teamcity.
> > >> >> > >
> > >> >> > >
> > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> [hidden email]
> > >
> > >> >> wrote:
> > >> >> > > >
> > >> >> > > > Hi Igniters,
> > >> >> > > >
> > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> our
> > >> new
> > >> >> tests
> > >> >> > > > scripts with a couple of Java builds. WDYT?
> > >> >> > > >
> > >> >> > > > Sincerely,
> > >> >> > > > Dmitriy Pavlov
> > >> >> > > >
> > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > >> < [hidden email] >:
> > >> >> > > >
> > >> >> > > >> Hi Ignite Developers,
> > >> >> > > >>
> > >> >> > > >> In a separate discussion, I've shared a log with all
> commits.
> > >> >> > > >>
> > >> >> > > >> As far as I can see, nobody removed commits from this sheet,
> > so
> > >> the
> > >> >> > > scope
> > >> >> > > >> of release will be discussed in another way: only explicitly
> > >> >> declared
> > >> >> > > >> commits will be cherry-picked.
> > >> >> > > >>
> > >> >> > > >> Sincerely,
> > >> >> > > >> Dmitriy Pavlov
> > >> >> > > >>
> > >> >> > >
> > >> >> > >
> > >> >>
> > >> >
> >
> >
> > --
> > Zhenya Stanilovsky
> >
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

dmagda
Yes, the Python example's fix makes sense - that's a usability issue and
doesn't affect the core components.

In general, I would suggest us to do a code freeze and push other changes
to another release. We can't include everything.

-
Denis


On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <[hidden email]>
wrote:

> Hello!
>
> How about adding https://issues.apache.org/jira/browse/IGNITE-11366 which
> fixes Python's release directory?
>
> I have stumbled on this issue personally.
>
> Regards,
> --
> Ilya Kasnacheev
>
>
> чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]>:
>
> > Dmitriy, the probability of having nodes down with these bugs under heavy
> > load conditions is pretty high. Good candidate for cherry-picking.
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > https://issues.apache.org/jira/browse/IGNITE-11127
> > -- Roman
> >
> >
> >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy Pavlov <
> > [hidden email]> wrote:
> >
> >  Hi,
> >
> > I've cherry-picked this commit. It seems it is critical because it also
> > fixes storage corruption.
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> <[hidden email]
> > >:
> >
> > > I suppose this ticket [1] : is very useful too.
> > >
> > >
> > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > > CorruptedTreeException during simultaneous cache put operations ]
> > >
> > > >
> > > >
> > > >------- Forwarded message -------
> > > >From: "Alexey Goncharuk" < [hidden email] >
> > > >To: dev < [hidden email] >
> > > >Cc:
> > > >Subject: Re: Ignite 2.7.5 Release scope
> > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > > >
> > > >Hello Ilya,
> > > >
> > > >I do not see any issues with the mentioned test. I see the following
> > > output
> > > >in the logs:
> > > >
> > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root]
> >>>
> > > >Stopping test:
> > >
> > >
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > >in 37768 ms <<<
> > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root]
> >>>
> > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root]
> >>>
> > > >Starting test class: IgniteClientConnectTest <<<
> > > >
> > > >The issue with Windows may be long connection timeouts, in this case
> we
> > > >should either split the suite into multiple ones or decrease the SPI
> > > >timeouts.
> > > >
> > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > [hidden email]
> > > >:
> > > >
> > > >> Hello!
> > > >>
> > > >> It seems that I can no longer test this case, on account of
> > > >>
> > > >>
> > >
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > >> hanging every time under Java 11 on Windows.
> > > >>
> > > >> Alexey, Ivan, can you please take a look?
> > > >>
> > > >>
> > > >>
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >>
> > > >> Regards,
> > > >>
> > > >> --
> > > >> Ilya Kasnacheev
> > > >>
> > > >>
> > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > [hidden email] >:
> > > >>
> > > >> > Hello!
> > > >> >
> > > >> > Basically there is a test that explicitly highlights this problem,
> > > that
> > > >> is
> > > >> > running SSL tests on Windows + Java 11. They will hang on Master
> but
> > > >> pass
> > > >> > with this patch.
> > > >> >
> > > >> > I have started that on TC, results will probably be available
> later
> > > >> today:
> > > >> >
> > > >> >
> > > >>
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > >> > (mind the Java version).
> > > >> >
> > > >> > Regards,
> > > >> > --
> > > >> > Ilya Kasnacheev
> > > >> >
> > > >> >
> > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov < [hidden email]
> > >:
> > > >> >
> > > >> >> Dmitry, Ilya,
> > > >> >>
> > > >> >> Yes, I've looked through those changes [1] as they can affect my
> > > local
> > > >> >> PR.  Basically, changes look good to me.
> > > >> >>
> > > >> >> I'm not an expert with CommunicationSpi component, so can miss
> some
> > > >> >> details and I haven't tested these changes under Java 11. One
> more
> > > >> >> thing I'd like to say, I would add additional tests to PR that
> will
> > > >> >> explicitly highlight the problem being solved.
> > > >> >>
> > > >> >>
> > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> > > >> >>
> > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> [hidden email]
> > >
> > > >> wrote:
> > > >> >> >
> > > >> >> > Hi Igniters,
> > > >> >> >
> > > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299 Avoid
> > busy
> > > >> wait
> > > >> >> on
> > > >> >> > processWrite during SSL handshake.
> > > >> >> > seems to be blocker cause it is related to Java 11
> > > >> >> >
> > > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > >> comments
> > > >> >> > addressed?
> > > >> >> >
> > > >> >> > The ticket is in Patch Available. Reviewer needed. Changes
> > located
> > > >> in
> > > >> >> > GridNioServer.
> > > >> >> >
> > > >> >> > Sincerely,
> > > >> >> > Dmitriy Pavlov
> > > >> >> >
> > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > > >> >> >
> > > >> >> >
> > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov < [hidden email]
> > >:
> > > >> >> >
> > > >> >> > > Huge +1
> > > >> >> > >
> > > >> >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > >> >> > >
> > > >> >> > >
> > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > [hidden email]
> > > >
> > > >> >> wrote:
> > > >> >> > > >
> > > >> >> > > > Hi Igniters,
> > > >> >> > > >
> > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at least test
> > our
> > > >> new
> > > >> >> tests
> > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > > >> >> > > >
> > > >> >> > > > Sincerely,
> > > >> >> > > > Dmitriy Pavlov
> > > >> >> > > >
> > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > > >> < [hidden email] >:
> > > >> >> > > >
> > > >> >> > > >> Hi Ignite Developers,
> > > >> >> > > >>
> > > >> >> > > >> In a separate discussion, I've shared a log with all
> > commits.
> > > >> >> > > >>
> > > >> >> > > >> As far as I can see, nobody removed commits from this
> sheet,
> > > so
> > > >> the
> > > >> >> > > scope
> > > >> >> > > >> of release will be discussed in another way: only
> explicitly
> > > >> >> declared
> > > >> >> > > >> commits will be cherry-picked.
> > > >> >> > > >>
> > > >> >> > > >> Sincerely,
> > > >> >> > > >> Dmitriy Pavlov
> > > >> >> > > >>
> > > >> >> > >
> > > >> >> > >
> > > >> >>
> > > >> >
> > >
> > >
> > > --
> > > Zhenya Stanilovsky
> > >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Denis, I also +1 for python fix. Today is the planned date of code freeze,
so today or tomorrow I'll announce it.

Only one thing was suggested is the fix for failure handlers. I don't
deeply understand the pros and cons, but I'm sure we should courteous
enough for wait experienced folks to come to a consensus.

One more thing I suggest to check after code freeze, it is Java 12. It does
not work now, but with thanks to Andrey G and Ivan P. we probably have not
very complex solution for this.

So these 2 fixes are last candidates to come to scope. WDYT?

Sincerely,
Dmitriy Pavlov

чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:

> Yes, the Python example's fix makes sense - that's a usability issue and
> doesn't affect the core components.
>
> In general, I would suggest us to do a code freeze and push other changes
> to another release. We can't include everything.
>
> -
> Denis
>
>
> On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <[hidden email]
> >
> wrote:
>
> > Hello!
> >
> > How about adding https://issues.apache.org/jira/browse/IGNITE-11366
> which
> > fixes Python's release directory?
> >
> > I have stumbled on this issue personally.
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]>:
> >
> > > Dmitriy, the probability of having nodes down with these bugs under
> heavy
> > > load conditions is pretty high. Good candidate for cherry-picking.
> > >
> > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > > https://issues.apache.org/jira/browse/IGNITE-11127
> > > -- Roman
> > >
> > >
> > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy Pavlov <
> > > [hidden email]> wrote:
> > >
> > >  Hi,
> > >
> > > I've cherry-picked this commit. It seems it is critical because it also
> > > fixes storage corruption.
> > >
> > > Sincerely,
> > > Dmitriy Pavlov
> > >
> > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> > <[hidden email]
> > > >:
> > >
> > > > I suppose this ticket [1] : is very useful too.
> > > >
> > > >
> > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > > > CorruptedTreeException during simultaneous cache put operations ]
> > > >
> > > > >
> > > > >
> > > > >------- Forwarded message -------
> > > > >From: "Alexey Goncharuk" < [hidden email] >
> > > > >To: dev < [hidden email] >
> > > > >Cc:
> > > > >Subject: Re: Ignite 2.7.5 Release scope
> > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > > > >
> > > > >Hello Ilya,
> > > > >
> > > > >I do not see any issues with the mentioned test. I see the following
> > > > output
> > > > >in the logs:
> > > > >
> > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO ][main][root]
> > >>>
> > > > >Stopping test:
> > > >
> > > >
> > >
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > > >in 37768 ms <<<
> > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root]
> > >>>
> > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO ][main][root]
> > >>>
> > > > >Starting test class: IgniteClientConnectTest <<<
> > > > >
> > > > >The issue with Windows may be long connection timeouts, in this case
> > we
> > > > >should either split the suite into multiple ones or decrease the SPI
> > > > >timeouts.
> > > > >
> > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > > [hidden email]
> > > > >:
> > > > >
> > > > >> Hello!
> > > > >>
> > > > >> It seems that I can no longer test this case, on account of
> > > > >>
> > > > >>
> > > >
> > >
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > > >> hanging every time under Java 11 on Windows.
> > > > >>
> > > > >> Alexey, Ivan, can you please take a look?
> > > > >>
> > > > >>
> > > > >>
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > >>
> > > > >> Regards,
> > > > >>
> > > > >> --
> > > > >> Ilya Kasnacheev
> > > > >>
> > > > >>
> > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > > [hidden email] >:
> > > > >>
> > > > >> > Hello!
> > > > >> >
> > > > >> > Basically there is a test that explicitly highlights this
> problem,
> > > > that
> > > > >> is
> > > > >> > running SSL tests on Windows + Java 11. They will hang on Master
> > but
> > > > >> pass
> > > > >> > with this patch.
> > > > >> >
> > > > >> > I have started that on TC, results will probably be available
> > later
> > > > >> today:
> > > > >> >
> > > > >> >
> > > > >>
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > >> > (mind the Java version).
> > > > >> >
> > > > >> > Regards,
> > > > >> > --
> > > > >> > Ilya Kasnacheev
> > > > >> >
> > > > >> >
> > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> [hidden email]
> > > >:
> > > > >> >
> > > > >> >> Dmitry, Ilya,
> > > > >> >>
> > > > >> >> Yes, I've looked through those changes [1] as they can affect
> my
> > > > local
> > > > >> >> PR.  Basically, changes look good to me.
> > > > >> >>
> > > > >> >> I'm not an expert with CommunicationSpi component, so can miss
> > some
> > > > >> >> details and I haven't tested these changes under Java 11. One
> > more
> > > > >> >> thing I'd like to say, I would add additional tests to PR that
> > will
> > > > >> >> explicitly highlight the problem being solved.
> > > > >> >>
> > > > >> >>
> > > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> > > > >> >>
> > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> > [hidden email]
> > > >
> > > > >> wrote:
> > > > >> >> >
> > > > >> >> > Hi Igniters,
> > > > >> >> >
> > > > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299
> Avoid
> > > busy
> > > > >> wait
> > > > >> >> on
> > > > >> >> > processWrite during SSL handshake.
> > > > >> >> > seems to be blocker cause it is related to Java 11
> > > > >> >> >
> > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were these
> > > > >> comments
> > > > >> >> > addressed?
> > > > >> >> >
> > > > >> >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > located
> > > > >> in
> > > > >> >> > GridNioServer.
> > > > >> >> >
> > > > >> >> > Sincerely,
> > > > >> >> > Dmitriy Pavlov
> > > > >> >> >
> > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > > > >> >> >
> > > > >> >> >
> > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> [hidden email]
> > > >:
> > > > >> >> >
> > > > >> >> > > Huge +1
> > > > >> >> > >
> > > > >> >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > > >> >> > >
> > > > >> >> > >
> > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > > [hidden email]
> > > > >
> > > > >> >> wrote:
> > > > >> >> > > >
> > > > >> >> > > > Hi Igniters,
> > > > >> >> > > >
> > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at least
> test
> > > our
> > > > >> new
> > > > >> >> tests
> > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > > > >> >> > > >
> > > > >> >> > > > Sincerely,
> > > > >> >> > > > Dmitriy Pavlov
> > > > >> >> > > >
> > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > > > >> < [hidden email] >:
> > > > >> >> > > >
> > > > >> >> > > >> Hi Ignite Developers,
> > > > >> >> > > >>
> > > > >> >> > > >> In a separate discussion, I've shared a log with all
> > > commits.
> > > > >> >> > > >>
> > > > >> >> > > >> As far as I can see, nobody removed commits from this
> > sheet,
> > > > so
> > > > >> the
> > > > >> >> > > scope
> > > > >> >> > > >> of release will be discussed in another way: only
> > explicitly
> > > > >> >> declared
> > > > >> >> > > >> commits will be cherry-picked.
> > > > >> >> > > >>
> > > > >> >> > > >> Sincerely,
> > > > >> >> > > >> Dmitriy Pavlov
> > > > >> >> > > >>
> > > > >> >> > >
> > > > >> >> > >
> > > > >> >>
> > > > >> >
> > > >
> > > >
> > > > --
> > > > Zhenya Stanilovsky
> > > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

dmagda
If the failure handler improvements will lower down a number of
false-positive cluster shutdowns then I'm for the fix inclusion to the
release.

As for Java 12, I would put to the next release that we can make shortly
after this one. We don't need to wait for months if there are some urgent
fixes.

-
Denis


On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]> wrote:

> Denis, I also +1 for python fix. Today is the planned date of code freeze,
> so today or tomorrow I'll announce it.
>
> Only one thing was suggested is the fix for failure handlers. I don't
> deeply understand the pros and cons, but I'm sure we should courteous
> enough for wait experienced folks to come to a consensus.
>
> One more thing I suggest to check after code freeze, it is Java 12. It does
> not work now, but with thanks to Andrey G and Ivan P. we probably have not
> very complex solution for this.
>
> So these 2 fixes are last candidates to come to scope. WDYT?
>
> Sincerely,
> Dmitriy Pavlov
>
> чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
>
> > Yes, the Python example's fix makes sense - that's a usability issue and
> > doesn't affect the core components.
> >
> > In general, I would suggest us to do a code freeze and push other changes
> > to another release. We can't include everything.
> >
> > -
> > Denis
> >
> >
> > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> [hidden email]
> > >
> > wrote:
> >
> > > Hello!
> > >
> > > How about adding https://issues.apache.org/jira/browse/IGNITE-11366
> > which
> > > fixes Python's release directory?
> > >
> > > I have stumbled on this issue personally.
> > >
> > > Regards,
> > > --
> > > Ilya Kasnacheev
> > >
> > >
> > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]>:
> > >
> > > > Dmitriy, the probability of having nodes down with these bugs under
> > heavy
> > > > load conditions is pretty high. Good candidate for cherry-picking.
> > > >
> > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > > > https://issues.apache.org/jira/browse/IGNITE-11127
> > > > -- Roman
> > > >
> > > >
> > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy Pavlov
> <
> > > > [hidden email]> wrote:
> > > >
> > > >  Hi,
> > > >
> > > > I've cherry-picked this commit. It seems it is critical because it
> also
> > > > fixes storage corruption.
> > > >
> > > > Sincerely,
> > > > Dmitriy Pavlov
> > > >
> > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> > > <[hidden email]
> > > > >:
> > > >
> > > > > I suppose this ticket [1] : is very useful too.
> > > > >
> > > > >
> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > > > > CorruptedTreeException during simultaneous cache put operations ]
> > > > >
> > > > > >
> > > > > >
> > > > > >------- Forwarded message -------
> > > > > >From: "Alexey Goncharuk" < [hidden email] >
> > > > > >To: dev < [hidden email] >
> > > > > >Cc:
> > > > > >Subject: Re: Ignite 2.7.5 Release scope
> > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > > > > >
> > > > > >Hello Ilya,
> > > > > >
> > > > > >I do not see any issues with the mentioned test. I see the
> following
> > > > > output
> > > > > >in the logs:
> > > > > >
> > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> ][main][root]
> > > >>>
> > > > > >Stopping test:
> > > > >
> > > > >
> > > >
> > >
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > > > >in 37768 ms <<<
> > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> ][main][root]
> > > >>>
> > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> ][main][root]
> > > >>>
> > > > > >Starting test class: IgniteClientConnectTest <<<
> > > > > >
> > > > > >The issue with Windows may be long connection timeouts, in this
> case
> > > we
> > > > > >should either split the suite into multiple ones or decrease the
> SPI
> > > > > >timeouts.
> > > > > >
> > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > > > [hidden email]
> > > > > >:
> > > > > >
> > > > > >> Hello!
> > > > > >>
> > > > > >> It seems that I can no longer test this case, on account of
> > > > > >>
> > > > > >>
> > > > >
> > > >
> > >
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > > > >> hanging every time under Java 11 on Windows.
> > > > > >>
> > > > > >> Alexey, Ivan, can you please take a look?
> > > > > >>
> > > > > >>
> > > > > >>
> > > > >
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > >>
> > > > > >> Regards,
> > > > > >>
> > > > > >> --
> > > > > >> Ilya Kasnacheev
> > > > > >>
> > > > > >>
> > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > > > [hidden email] >:
> > > > > >>
> > > > > >> > Hello!
> > > > > >> >
> > > > > >> > Basically there is a test that explicitly highlights this
> > problem,
> > > > > that
> > > > > >> is
> > > > > >> > running SSL tests on Windows + Java 11. They will hang on
> Master
> > > but
> > > > > >> pass
> > > > > >> > with this patch.
> > > > > >> >
> > > > > >> > I have started that on TC, results will probably be available
> > > later
> > > > > >> today:
> > > > > >> >
> > > > > >> >
> > > > > >>
> > > > >
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > >> > (mind the Java version).
> > > > > >> >
> > > > > >> > Regards,
> > > > > >> > --
> > > > > >> > Ilya Kasnacheev
> > > > > >> >
> > > > > >> >
> > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> > [hidden email]
> > > > >:
> > > > > >> >
> > > > > >> >> Dmitry, Ilya,
> > > > > >> >>
> > > > > >> >> Yes, I've looked through those changes [1] as they can affect
> > my
> > > > > local
> > > > > >> >> PR.  Basically, changes look good to me.
> > > > > >> >>
> > > > > >> >> I'm not an expert with CommunicationSpi component, so can
> miss
> > > some
> > > > > >> >> details and I haven't tested these changes under Java 11. One
> > > more
> > > > > >> >> thing I'd like to say, I would add additional tests to PR
> that
> > > will
> > > > > >> >> explicitly highlight the problem being solved.
> > > > > >> >>
> > > > > >> >>
> > > > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> > > > > >> >>
> > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> > > [hidden email]
> > > > >
> > > > > >> wrote:
> > > > > >> >> >
> > > > > >> >> > Hi Igniters,
> > > > > >> >> >
> > > > > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299
> > Avoid
> > > > busy
> > > > > >> wait
> > > > > >> >> on
> > > > > >> >> > processWrite during SSL handshake.
> > > > > >> >> > seems to be blocker cause it is related to Java 11
> > > > > >> >> >
> > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were
> these
> > > > > >> comments
> > > > > >> >> > addressed?
> > > > > >> >> >
> > > > > >> >> > The ticket is in Patch Available. Reviewer needed. Changes
> > > > located
> > > > > >> in
> > > > > >> >> > GridNioServer.
> > > > > >> >> >
> > > > > >> >> > Sincerely,
> > > > > >> >> > Dmitriy Pavlov
> > > > > >> >> >
> > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > > > > >> >> >
> > > > > >> >> >
> > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> > [hidden email]
> > > > >:
> > > > > >> >> >
> > > > > >> >> > > Huge +1
> > > > > >> >> > >
> > > > > >> >> > > Will try to add new JDK in nearest time to our Teamcity.
> > > > > >> >> > >
> > > > > >> >> > >
> > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > > > [hidden email]
> > > > > >
> > > > > >> >> wrote:
> > > > > >> >> > > >
> > > > > >> >> > > > Hi Igniters,
> > > > > >> >> > > >
> > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at least
> > test
> > > > our
> > > > > >> new
> > > > > >> >> tests
> > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > > > > >> >> > > >
> > > > > >> >> > > > Sincerely,
> > > > > >> >> > > > Dmitriy Pavlov
> > > > > >> >> > > >
> > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > > > > >> < [hidden email] >:
> > > > > >> >> > > >
> > > > > >> >> > > >> Hi Ignite Developers,
> > > > > >> >> > > >>
> > > > > >> >> > > >> In a separate discussion, I've shared a log with all
> > > > commits.
> > > > > >> >> > > >>
> > > > > >> >> > > >> As far as I can see, nobody removed commits from this
> > > sheet,
> > > > > so
> > > > > >> the
> > > > > >> >> > > scope
> > > > > >> >> > > >> of release will be discussed in another way: only
> > > explicitly
> > > > > >> >> declared
> > > > > >> >> > > >> commits will be cherry-picked.
> > > > > >> >> > > >>
> > > > > >> >> > > >> Sincerely,
> > > > > >> >> > > >> Dmitriy Pavlov
> > > > > >> >> > > >>
> > > > > >> >> > >
> > > > > >> >> > >
> > > > > >> >>
> > > > > >> >
> > > > >
> > > > >
> > > > > --
> > > > > Zhenya Stanilovsky
> > > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Hi Denis,

I'm not talking about months. In this discussion, Andrey and Ivan proposed
a couple of fixes that may help.

It will require a day or two to check if it helps. If it not helpful then
we should modify startup scripts to say clearly that Java 12 is not
supported.

Now under Java 12 Ignite suggests to set startup parameters, but even with
correct parameters, it fails and says please set parameters. Totally
unclear for end-user.

I've reopened https://issues.apache.org/jira/browse/IGNITE-11600

Sincerely,
Dmitriy Pavlov

чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:

> If the failure handler improvements will lower down a number of
> false-positive cluster shutdowns then I'm for the fix inclusion to the
> release.
>
> As for Java 12, I would put to the next release that we can make shortly
> after this one. We don't need to wait for months if there are some urgent
> fixes.
>
> -
> Denis
>
>
> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]> wrote:
>
> > Denis, I also +1 for python fix. Today is the planned date of code
> freeze,
> > so today or tomorrow I'll announce it.
> >
> > Only one thing was suggested is the fix for failure handlers. I don't
> > deeply understand the pros and cons, but I'm sure we should courteous
> > enough for wait experienced folks to come to a consensus.
> >
> > One more thing I suggest to check after code freeze, it is Java 12. It
> does
> > not work now, but with thanks to Andrey G and Ivan P. we probably have
> not
> > very complex solution for this.
> >
> > So these 2 fixes are last candidates to come to scope. WDYT?
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
> >
> > > Yes, the Python example's fix makes sense - that's a usability issue
> and
> > > doesn't affect the core components.
> > >
> > > In general, I would suggest us to do a code freeze and push other
> changes
> > > to another release. We can't include everything.
> > >
> > > -
> > > Denis
> > >
> > >
> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> > [hidden email]
> > > >
> > > wrote:
> > >
> > > > Hello!
> > > >
> > > > How about adding https://issues.apache.org/jira/browse/IGNITE-11366
> > > which
> > > > fixes Python's release directory?
> > > >
> > > > I have stumbled on this issue personally.
> > > >
> > > > Regards,
> > > > --
> > > > Ilya Kasnacheev
> > > >
> > > >
> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]
> >:
> > > >
> > > > > Dmitriy, the probability of having nodes down with these bugs under
> > > heavy
> > > > > load conditions is pretty high. Good candidate for cherry-picking.
> > > > >
> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
> > > > > -- Roman
> > > > >
> > > > >
> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy
> Pavlov
> > <
> > > > > [hidden email]> wrote:
> > > > >
> > > > >  Hi,
> > > > >
> > > > > I've cherry-picked this commit. It seems it is critical because it
> > also
> > > > > fixes storage corruption.
> > > > >
> > > > > Sincerely,
> > > > > Dmitriy Pavlov
> > > > >
> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> > > > <[hidden email]
> > > > > >:
> > > > >
> > > > > > I suppose this ticket [1] : is very useful too.
> > > > > >
> > > > > >
> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > > > > > CorruptedTreeException during simultaneous cache put operations ]
> > > > > >
> > > > > > >
> > > > > > >
> > > > > > >------- Forwarded message -------
> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
> > > > > > >To: dev < [hidden email] >
> > > > > > >Cc:
> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > > > > > >
> > > > > > >Hello Ilya,
> > > > > > >
> > > > > > >I do not see any issues with the mentioned test. I see the
> > following
> > > > > > output
> > > > > > >in the logs:
> > > > > > >
> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> > ][main][root]
> > > > >>>
> > > > > > >Stopping test:
> > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > > > > > >in 37768 ms <<<
> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > ][main][root]
> > > > >>>
> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > ][main][root]
> > > > >>>
> > > > > > >Starting test class: IgniteClientConnectTest <<<
> > > > > > >
> > > > > > >The issue with Windows may be long connection timeouts, in this
> > case
> > > > we
> > > > > > >should either split the suite into multiple ones or decrease the
> > SPI
> > > > > > >timeouts.
> > > > > > >
> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > > > > [hidden email]
> > > > > > >:
> > > > > > >
> > > > > > >> Hello!
> > > > > > >>
> > > > > > >> It seems that I can no longer test this case, on account of
> > > > > > >>
> > > > > > >>
> > > > > >
> > > > >
> > > >
> > >
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > > > > > >> hanging every time under Java 11 on Windows.
> > > > > > >>
> > > > > > >> Alexey, Ivan, can you please take a look?
> > > > > > >>
> > > > > > >>
> > > > > > >>
> > > > > >
> > > > >
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > > >>
> > > > > > >> Regards,
> > > > > > >>
> > > > > > >> --
> > > > > > >> Ilya Kasnacheev
> > > > > > >>
> > > > > > >>
> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > > > > > [hidden email] >:
> > > > > > >>
> > > > > > >> > Hello!
> > > > > > >> >
> > > > > > >> > Basically there is a test that explicitly highlights this
> > > problem,
> > > > > > that
> > > > > > >> is
> > > > > > >> > running SSL tests on Windows + Java 11. They will hang on
> > Master
> > > > but
> > > > > > >> pass
> > > > > > >> > with this patch.
> > > > > > >> >
> > > > > > >> > I have started that on TC, results will probably be
> available
> > > > later
> > > > > > >> today:
> > > > > > >> >
> > > > > > >> >
> > > > > > >>
> > > > > >
> > > > >
> > > >
> > >
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > > > > > >> > (mind the Java version).
> > > > > > >> >
> > > > > > >> > Regards,
> > > > > > >> > --
> > > > > > >> > Ilya Kasnacheev
> > > > > > >> >
> > > > > > >> >
> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> > > [hidden email]
> > > > > >:
> > > > > > >> >
> > > > > > >> >> Dmitry, Ilya,
> > > > > > >> >>
> > > > > > >> >> Yes, I've looked through those changes [1] as they can
> affect
> > > my
> > > > > > local
> > > > > > >> >> PR.  Basically, changes look good to me.
> > > > > > >> >>
> > > > > > >> >> I'm not an expert with CommunicationSpi component, so can
> > miss
> > > > some
> > > > > > >> >> details and I haven't tested these changes under Java 11.
> One
> > > > more
> > > > > > >> >> thing I'd like to say, I would add additional tests to PR
> > that
> > > > will
> > > > > > >> >> explicitly highlight the problem being solved.
> > > > > > >> >>
> > > > > > >> >>
> > > > > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> > > > > > >> >>
> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> > > > [hidden email]
> > > > > >
> > > > > > >> wrote:
> > > > > > >> >> >
> > > > > > >> >> > Hi Igniters,
> > > > > > >> >> >
> > > > > > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299
> > > Avoid
> > > > > busy
> > > > > > >> wait
> > > > > > >> >> on
> > > > > > >> >> > processWrite during SSL handshake.
> > > > > > >> >> > seems to be blocker cause it is related to Java 11
> > > > > > >> >> >
> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were
> > these
> > > > > > >> comments
> > > > > > >> >> > addressed?
> > > > > > >> >> >
> > > > > > >> >> > The ticket is in Patch Available. Reviewer needed.
> Changes
> > > > > located
> > > > > > >> in
> > > > > > >> >> > GridNioServer.
> > > > > > >> >> >
> > > > > > >> >> > Sincerely,
> > > > > > >> >> > Dmitriy Pavlov
> > > > > > >> >> >
> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > > > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > > > > > >> >> >
> > > > > > >> >> >
> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> > > [hidden email]
> > > > > >:
> > > > > > >> >> >
> > > > > > >> >> > > Huge +1
> > > > > > >> >> > >
> > > > > > >> >> > > Will try to add new JDK in nearest time to our
> Teamcity.
> > > > > > >> >> > >
> > > > > > >> >> > >
> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > > > > [hidden email]
> > > > > > >
> > > > > > >> >> wrote:
> > > > > > >> >> > > >
> > > > > > >> >> > > > Hi Igniters,
> > > > > > >> >> > > >
> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at
> least
> > > test
> > > > > our
> > > > > > >> new
> > > > > > >> >> tests
> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > > > > > >> >> > > >
> > > > > > >> >> > > > Sincerely,
> > > > > > >> >> > > > Dmitriy Pavlov
> > > > > > >> >> > > >
> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > > > > > >> < [hidden email] >:
> > > > > > >> >> > > >
> > > > > > >> >> > > >> Hi Ignite Developers,
> > > > > > >> >> > > >>
> > > > > > >> >> > > >> In a separate discussion, I've shared a log with all
> > > > > commits.
> > > > > > >> >> > > >>
> > > > > > >> >> > > >> As far as I can see, nobody removed commits from
> this
> > > > sheet,
> > > > > > so
> > > > > > >> the
> > > > > > >> >> > > scope
> > > > > > >> >> > > >> of release will be discussed in another way: only
> > > > explicitly
> > > > > > >> >> declared
> > > > > > >> >> > > >> commits will be cherry-picked.
> > > > > > >> >> > > >>
> > > > > > >> >> > > >> Sincerely,
> > > > > > >> >> > > >> Dmitriy Pavlov
> > > > > > >> >> > > >>
> > > > > > >> >> > >
> > > > > > >> >> > >
> > > > > > >> >>
> > > > > > >> >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Zhenya Stanilovsky
> > > > > >
> > > >
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Hi Igniters,

 I would like to announce code freeze for 2.7.5. Only one open ticket is
there (reopened): https://issues.apache.org/jira/browse/IGNITE-11600 (if we
can't start using Java 12 we should clearly state it in scripts/code).

We're entering to Stabilization phase for
https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5 and
only blockers may be included into scope. See
https://cwiki.apache.org/confluence/display/IGNITE/Release+Process for more
details.

Sincerely,
Dmitriy Pavlov

пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:

> Hi Denis,
>
> I'm not talking about months. In this discussion, Andrey and Ivan proposed
> a couple of fixes that may help.
>
> It will require a day or two to check if it helps. If it not helpful then
> we should modify startup scripts to say clearly that Java 12 is not
> supported.
>
> Now under Java 12 Ignite suggests to set startup parameters, but even with
> correct parameters, it fails and says please set parameters. Totally
> unclear for end-user.
>
> I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
>
> Sincerely,
> Dmitriy Pavlov
>
> чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
>
>> If the failure handler improvements will lower down a number of
>> false-positive cluster shutdowns then I'm for the fix inclusion to the
>> release.
>>
>> As for Java 12, I would put to the next release that we can make shortly
>> after this one. We don't need to wait for months if there are some urgent
>> fixes.
>>
>> -
>> Denis
>>
>>
>> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]>
>> wrote:
>>
>> > Denis, I also +1 for python fix. Today is the planned date of code
>> freeze,
>> > so today or tomorrow I'll announce it.
>> >
>> > Only one thing was suggested is the fix for failure handlers. I don't
>> > deeply understand the pros and cons, but I'm sure we should courteous
>> > enough for wait experienced folks to come to a consensus.
>> >
>> > One more thing I suggest to check after code freeze, it is Java 12. It
>> does
>> > not work now, but with thanks to Andrey G and Ivan P. we probably have
>> not
>> > very complex solution for this.
>> >
>> > So these 2 fixes are last candidates to come to scope. WDYT?
>> >
>> > Sincerely,
>> > Dmitriy Pavlov
>> >
>> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
>> >
>> > > Yes, the Python example's fix makes sense - that's a usability issue
>> and
>> > > doesn't affect the core components.
>> > >
>> > > In general, I would suggest us to do a code freeze and push other
>> changes
>> > > to another release. We can't include everything.
>> > >
>> > > -
>> > > Denis
>> > >
>> > >
>> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
>> > [hidden email]
>> > > >
>> > > wrote:
>> > >
>> > > > Hello!
>> > > >
>> > > > How about adding https://issues.apache.org/jira/browse/IGNITE-11366
>> > > which
>> > > > fixes Python's release directory?
>> > > >
>> > > > I have stumbled on this issue personally.
>> > > >
>> > > > Regards,
>> > > > --
>> > > > Ilya Kasnacheev
>> > > >
>> > > >
>> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh <[hidden email]
>> >:
>> > > >
>> > > > > Dmitriy, the probability of having nodes down with these bugs
>> under
>> > > heavy
>> > > > > load conditions is pretty high. Good candidate for cherry-picking.
>> > > > >
>> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
>> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
>> > > > > -- Roman
>> > > > >
>> > > > >
>> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy
>> Pavlov
>> > <
>> > > > > [hidden email]> wrote:
>> > > > >
>> > > > >  Hi,
>> > > > >
>> > > > > I've cherry-picked this commit. It seems it is critical because it
>> > also
>> > > > > fixes storage corruption.
>> > > > >
>> > > > > Sincerely,
>> > > > > Dmitriy Pavlov
>> > > > >
>> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
>> > > > <[hidden email]
>> > > > > >:
>> > > > >
>> > > > > > I suppose this ticket [1] : is very useful too.
>> > > > > >
>> > > > > >
>> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
>> > > > > > CorruptedTreeException during simultaneous cache put operations
>> ]
>> > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > >------- Forwarded message -------
>> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
>> > > > > > >To: dev < [hidden email] >
>> > > > > > >Cc:
>> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
>> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
>> > > > > > >
>> > > > > > >Hello Ilya,
>> > > > > > >
>> > > > > > >I do not see any issues with the mentioned test. I see the
>> > following
>> > > > > > output
>> > > > > > >in the logs:
>> > > > > > >
>> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
>> > ][main][root]
>> > > > >>>
>> > > > > > >Stopping test:
>> > > > > >
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
>> > > > > > >in 37768 ms <<<
>> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
>> > ][main][root]
>> > > > >>>
>> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
>> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
>> > ][main][root]
>> > > > >>>
>> > > > > > >Starting test class: IgniteClientConnectTest <<<
>> > > > > > >
>> > > > > > >The issue with Windows may be long connection timeouts, in this
>> > case
>> > > > we
>> > > > > > >should either split the suite into multiple ones or decrease
>> the
>> > SPI
>> > > > > > >timeouts.
>> > > > > > >
>> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
>> > > > > [hidden email]
>> > > > > > >:
>> > > > > > >
>> > > > > > >> Hello!
>> > > > > > >>
>> > > > > > >> It seems that I can no longer test this case, on account of
>> > > > > > >>
>> > > > > > >>
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
>> > > > > > >> hanging every time under Java 11 on Windows.
>> > > > > > >>
>> > > > > > >> Alexey, Ivan, can you please take a look?
>> > > > > > >>
>> > > > > > >>
>> > > > > > >>
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > > > > > >>
>> > > > > > >> Regards,
>> > > > > > >>
>> > > > > > >> --
>> > > > > > >> Ilya Kasnacheev
>> > > > > > >>
>> > > > > > >>
>> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
>> > > > > > [hidden email] >:
>> > > > > > >>
>> > > > > > >> > Hello!
>> > > > > > >> >
>> > > > > > >> > Basically there is a test that explicitly highlights this
>> > > problem,
>> > > > > > that
>> > > > > > >> is
>> > > > > > >> > running SSL tests on Windows + Java 11. They will hang on
>> > Master
>> > > > but
>> > > > > > >> pass
>> > > > > > >> > with this patch.
>> > > > > > >> >
>> > > > > > >> > I have started that on TC, results will probably be
>> available
>> > > > later
>> > > > > > >> today:
>> > > > > > >> >
>> > > > > > >> >
>> > > > > > >>
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > > > > > >> > (mind the Java version).
>> > > > > > >> >
>> > > > > > >> > Regards,
>> > > > > > >> > --
>> > > > > > >> > Ilya Kasnacheev
>> > > > > > >> >
>> > > > > > >> >
>> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
>> > > [hidden email]
>> > > > > >:
>> > > > > > >> >
>> > > > > > >> >> Dmitry, Ilya,
>> > > > > > >> >>
>> > > > > > >> >> Yes, I've looked through those changes [1] as they can
>> affect
>> > > my
>> > > > > > local
>> > > > > > >> >> PR.  Basically, changes look good to me.
>> > > > > > >> >>
>> > > > > > >> >> I'm not an expert with CommunicationSpi component, so can
>> > miss
>> > > > some
>> > > > > > >> >> details and I haven't tested these changes under Java 11.
>> One
>> > > > more
>> > > > > > >> >> thing I'd like to say, I would add additional tests to PR
>> > that
>> > > > will
>> > > > > > >> >> explicitly highlight the problem being solved.
>> > > > > > >> >>
>> > > > > > >> >>
>> > > > > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
>> > > > > > >> >>
>> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
>> > > > [hidden email]
>> > > > > >
>> > > > > > >> wrote:
>> > > > > > >> >> >
>> > > > > > >> >> > Hi Igniters,
>> > > > > > >> >> >
>> > > > > > >> >> > fix  https://issues.apache.org/jira/browse/IGNITE-11299
>> > > Avoid
>> > > > > busy
>> > > > > > >> wait
>> > > > > > >> >> on
>> > > > > > >> >> > processWrite during SSL handshake.
>> > > > > > >> >> > seems to be blocker cause it is related to Java 11
>> > > > > > >> >> >
>> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim M.were
>> > these
>> > > > > > >> comments
>> > > > > > >> >> > addressed?
>> > > > > > >> >> >
>> > > > > > >> >> > The ticket is in Patch Available. Reviewer needed.
>> Changes
>> > > > > located
>> > > > > > >> in
>> > > > > > >> >> > GridNioServer.
>> > > > > > >> >> >
>> > > > > > >> >> > Sincerely,
>> > > > > > >> >> > Dmitriy Pavlov
>> > > > > > >> >> >
>> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
>> > > > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
>> > > > > > >> >> >
>> > > > > > >> >> >
>> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
>> > > [hidden email]
>> > > > > >:
>> > > > > > >> >> >
>> > > > > > >> >> > > Huge +1
>> > > > > > >> >> > >
>> > > > > > >> >> > > Will try to add new JDK in nearest time to our
>> Teamcity.
>> > > > > > >> >> > >
>> > > > > > >> >> > >
>> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
>> > > > > [hidden email]
>> > > > > > >
>> > > > > > >> >> wrote:
>> > > > > > >> >> > > >
>> > > > > > >> >> > > > Hi Igniters,
>> > > > > > >> >> > > >
>> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at
>> least
>> > > test
>> > > > > our
>> > > > > > >> new
>> > > > > > >> >> tests
>> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
>> > > > > > >> >> > > >
>> > > > > > >> >> > > > Sincerely,
>> > > > > > >> >> > > > Dmitriy Pavlov
>> > > > > > >> >> > > >
>> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
>> > > > > > >> < [hidden email] >:
>> > > > > > >> >> > > >
>> > > > > > >> >> > > >> Hi Ignite Developers,
>> > > > > > >> >> > > >>
>> > > > > > >> >> > > >> In a separate discussion, I've shared a log with
>> all
>> > > > > commits.
>> > > > > > >> >> > > >>
>> > > > > > >> >> > > >> As far as I can see, nobody removed commits from
>> this
>> > > > sheet,
>> > > > > > so
>> > > > > > >> the
>> > > > > > >> >> > > scope
>> > > > > > >> >> > > >> of release will be discussed in another way: only
>> > > > explicitly
>> > > > > > >> >> declared
>> > > > > > >> >> > > >> commits will be cherry-picked.
>> > > > > > >> >> > > >>
>> > > > > > >> >> > > >> Sincerely,
>> > > > > > >> >> > > >> Dmitriy Pavlov
>> > > > > > >> >> > > >>
>> > > > > > >> >> > >
>> > > > > > >> >> > >
>> > > > > > >> >>
>> > > > > > >> >
>> > > > > >
>> > > > > >
>> > > > > > --
>> > > > > > Zhenya Stanilovsky
>> > > > > >
>> > > >
>> > >
>> >
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

dmagda
Folks,

What are the efforts to support Java 12? Let's do 2.7.6 shortly if the
fixes are time-consuming.

-
Denis


On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]> wrote:

> Hi Igniters,
>
>  I would like to announce code freeze for 2.7.5. Only one open ticket is
> there (reopened): https://issues.apache.org/jira/browse/IGNITE-11600 (if
> we
> can't start using Java 12 we should clearly state it in scripts/code).
>
> We're entering to Stabilization phase for
> https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5 and
> only blockers may be included into scope. See
> https://cwiki.apache.org/confluence/display/IGNITE/Release+Process for
> more
> details.
>
> Sincerely,
> Dmitriy Pavlov
>
> пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
>
> > Hi Denis,
> >
> > I'm not talking about months. In this discussion, Andrey and Ivan
> proposed
> > a couple of fixes that may help.
> >
> > It will require a day or two to check if it helps. If it not helpful then
> > we should modify startup scripts to say clearly that Java 12 is not
> > supported.
> >
> > Now under Java 12 Ignite suggests to set startup parameters, but even
> with
> > correct parameters, it fails and says please set parameters. Totally
> > unclear for end-user.
> >
> > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
> >
> >> If the failure handler improvements will lower down a number of
> >> false-positive cluster shutdowns then I'm for the fix inclusion to the
> >> release.
> >>
> >> As for Java 12, I would put to the next release that we can make shortly
> >> after this one. We don't need to wait for months if there are some
> urgent
> >> fixes.
> >>
> >> -
> >> Denis
> >>
> >>
> >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]>
> >> wrote:
> >>
> >> > Denis, I also +1 for python fix. Today is the planned date of code
> >> freeze,
> >> > so today or tomorrow I'll announce it.
> >> >
> >> > Only one thing was suggested is the fix for failure handlers. I don't
> >> > deeply understand the pros and cons, but I'm sure we should courteous
> >> > enough for wait experienced folks to come to a consensus.
> >> >
> >> > One more thing I suggest to check after code freeze, it is Java 12. It
> >> does
> >> > not work now, but with thanks to Andrey G and Ivan P. we probably have
> >> not
> >> > very complex solution for this.
> >> >
> >> > So these 2 fixes are last candidates to come to scope. WDYT?
> >> >
> >> > Sincerely,
> >> > Dmitriy Pavlov
> >> >
> >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
> >> >
> >> > > Yes, the Python example's fix makes sense - that's a usability issue
> >> and
> >> > > doesn't affect the core components.
> >> > >
> >> > > In general, I would suggest us to do a code freeze and push other
> >> changes
> >> > > to another release. We can't include everything.
> >> > >
> >> > > -
> >> > > Denis
> >> > >
> >> > >
> >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> >> > [hidden email]
> >> > > >
> >> > > wrote:
> >> > >
> >> > > > Hello!
> >> > > >
> >> > > > How about adding
> https://issues.apache.org/jira/browse/IGNITE-11366
> >> > > which
> >> > > > fixes Python's release directory?
> >> > > >
> >> > > > I have stumbled on this issue personally.
> >> > > >
> >> > > > Regards,
> >> > > > --
> >> > > > Ilya Kasnacheev
> >> > > >
> >> > > >
> >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
> <[hidden email]
> >> >:
> >> > > >
> >> > > > > Dmitriy, the probability of having nodes down with these bugs
> >> under
> >> > > heavy
> >> > > > > load conditions is pretty high. Good candidate for
> cherry-picking.
> >> > > > >
> >> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
> >> > > > > -- Roman
> >> > > > >
> >> > > > >
> >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy
> >> Pavlov
> >> > <
> >> > > > > [hidden email]> wrote:
> >> > > > >
> >> > > > >  Hi,
> >> > > > >
> >> > > > > I've cherry-picked this commit. It seems it is critical because
> it
> >> > also
> >> > > > > fixes storage corruption.
> >> > > > >
> >> > > > > Sincerely,
> >> > > > > Dmitriy Pavlov
> >> > > > >
> >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> >> > > > <[hidden email]
> >> > > > > >:
> >> > > > >
> >> > > > > > I suppose this ticket [1] : is very useful too.
> >> > > > > >
> >> > > > > >
> >> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> >> > > > > > CorruptedTreeException during simultaneous cache put
> operations
> >> ]
> >> > > > > >
> >> > > > > > >
> >> > > > > > >
> >> > > > > > >------- Forwarded message -------
> >> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
> >> > > > > > >To: dev < [hidden email] >
> >> > > > > > >Cc:
> >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
> >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> >> > > > > > >
> >> > > > > > >Hello Ilya,
> >> > > > > > >
> >> > > > > > >I do not see any issues with the mentioned test. I see the
> >> > following
> >> > > > > > output
> >> > > > > > >in the logs:
> >> > > > > > >
> >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> >> > ][main][root]
> >> > > > >>>
> >> > > > > > >Stopping test:
> >> > > > > >
> >> > > > > >
> >> > > > >
> >> > > >
> >> > >
> >> >
> >>
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> >> > > > > > >in 37768 ms <<<
> >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> >> > ][main][root]
> >> > > > >>>
> >> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> >> > ][main][root]
> >> > > > >>>
> >> > > > > > >Starting test class: IgniteClientConnectTest <<<
> >> > > > > > >
> >> > > > > > >The issue with Windows may be long connection timeouts, in
> this
> >> > case
> >> > > > we
> >> > > > > > >should either split the suite into multiple ones or decrease
> >> the
> >> > SPI
> >> > > > > > >timeouts.
> >> > > > > > >
> >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> >> > > > > [hidden email]
> >> > > > > > >:
> >> > > > > > >
> >> > > > > > >> Hello!
> >> > > > > > >>
> >> > > > > > >> It seems that I can no longer test this case, on account of
> >> > > > > > >>
> >> > > > > > >>
> >> > > > > >
> >> > > > >
> >> > > >
> >> > >
> >> >
> >>
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> >> > > > > > >> hanging every time under Java 11 on Windows.
> >> > > > > > >>
> >> > > > > > >> Alexey, Ivan, can you please take a look?
> >> > > > > > >>
> >> > > > > > >>
> >> > > > > > >>
> >> > > > > >
> >> > > > >
> >> > > >
> >> > >
> >> >
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >> > > > > > >>
> >> > > > > > >> Regards,
> >> > > > > > >>
> >> > > > > > >> --
> >> > > > > > >> Ilya Kasnacheev
> >> > > > > > >>
> >> > > > > > >>
> >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> >> > > > > > [hidden email] >:
> >> > > > > > >>
> >> > > > > > >> > Hello!
> >> > > > > > >> >
> >> > > > > > >> > Basically there is a test that explicitly highlights this
> >> > > problem,
> >> > > > > > that
> >> > > > > > >> is
> >> > > > > > >> > running SSL tests on Windows + Java 11. They will hang on
> >> > Master
> >> > > > but
> >> > > > > > >> pass
> >> > > > > > >> > with this patch.
> >> > > > > > >> >
> >> > > > > > >> > I have started that on TC, results will probably be
> >> available
> >> > > > later
> >> > > > > > >> today:
> >> > > > > > >> >
> >> > > > > > >> >
> >> > > > > > >>
> >> > > > > >
> >> > > > >
> >> > > >
> >> > >
> >> >
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >> > > > > > >> > (mind the Java version).
> >> > > > > > >> >
> >> > > > > > >> > Regards,
> >> > > > > > >> > --
> >> > > > > > >> > Ilya Kasnacheev
> >> > > > > > >> >
> >> > > > > > >> >
> >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> >> > > [hidden email]
> >> > > > > >:
> >> > > > > > >> >
> >> > > > > > >> >> Dmitry, Ilya,
> >> > > > > > >> >>
> >> > > > > > >> >> Yes, I've looked through those changes [1] as they can
> >> affect
> >> > > my
> >> > > > > > local
> >> > > > > > >> >> PR.  Basically, changes look good to me.
> >> > > > > > >> >>
> >> > > > > > >> >> I'm not an expert with CommunicationSpi component, so
> can
> >> > miss
> >> > > > some
> >> > > > > > >> >> details and I haven't tested these changes under Java
> 11.
> >> One
> >> > > > more
> >> > > > > > >> >> thing I'd like to say, I would add additional tests to
> PR
> >> > that
> >> > > > will
> >> > > > > > >> >> explicitly highlight the problem being solved.
> >> > > > > > >> >>
> >> > > > > > >> >>
> >> > > > > > >> >> [1]  https://issues.apache.org/jira/browse/IGNITE-11299
> >> > > > > > >> >>
> >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> >> > > > [hidden email]
> >> > > > > >
> >> > > > > > >> wrote:
> >> > > > > > >> >> >
> >> > > > > > >> >> > Hi Igniters,
> >> > > > > > >> >> >
> >> > > > > > >> >> > fix
> https://issues.apache.org/jira/browse/IGNITE-11299
> >> > > Avoid
> >> > > > > busy
> >> > > > > > >> wait
> >> > > > > > >> >> on
> >> > > > > > >> >> > processWrite during SSL handshake.
> >> > > > > > >> >> > seems to be blocker cause it is related to Java 11
> >> > > > > > >> >> >
> >> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim
> M.were
> >> > these
> >> > > > > > >> comments
> >> > > > > > >> >> > addressed?
> >> > > > > > >> >> >
> >> > > > > > >> >> > The ticket is in Patch Available. Reviewer needed.
> >> Changes
> >> > > > > located
> >> > > > > > >> in
> >> > > > > > >> >> > GridNioServer.
> >> > > > > > >> >> >
> >> > > > > > >> >> > Sincerely,
> >> > > > > > >> >> > Dmitriy Pavlov
> >> > > > > > >> >> >
> >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> >> > > > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> >> > > > > > >> >> >
> >> > > > > > >> >> >
> >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> >> > > [hidden email]
> >> > > > > >:
> >> > > > > > >> >> >
> >> > > > > > >> >> > > Huge +1
> >> > > > > > >> >> > >
> >> > > > > > >> >> > > Will try to add new JDK in nearest time to our
> >> Teamcity.
> >> > > > > > >> >> > >
> >> > > > > > >> >> > >
> >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> >> > > > > [hidden email]
> >> > > > > > >
> >> > > > > > >> >> wrote:
> >> > > > > > >> >> > > >
> >> > > > > > >> >> > > > Hi Igniters,
> >> > > > > > >> >> > > >
> >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at
> >> least
> >> > > test
> >> > > > > our
> >> > > > > > >> new
> >> > > > > > >> >> tests
> >> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> >> > > > > > >> >> > > >
> >> > > > > > >> >> > > > Sincerely,
> >> > > > > > >> >> > > > Dmitriy Pavlov
> >> > > > > > >> >> > > >
> >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> >> > > > > > >> < [hidden email] >:
> >> > > > > > >> >> > > >
> >> > > > > > >> >> > > >> Hi Ignite Developers,
> >> > > > > > >> >> > > >>
> >> > > > > > >> >> > > >> In a separate discussion, I've shared a log with
> >> all
> >> > > > > commits.
> >> > > > > > >> >> > > >>
> >> > > > > > >> >> > > >> As far as I can see, nobody removed commits from
> >> this
> >> > > > sheet,
> >> > > > > > so
> >> > > > > > >> the
> >> > > > > > >> >> > > scope
> >> > > > > > >> >> > > >> of release will be discussed in another way: only
> >> > > > explicitly
> >> > > > > > >> >> declared
> >> > > > > > >> >> > > >> commits will be cherry-picked.
> >> > > > > > >> >> > > >>
> >> > > > > > >> >> > > >> Sincerely,
> >> > > > > > >> >> > > >> Dmitriy Pavlov
> >> > > > > > >> >> > > >>
> >> > > > > > >> >> > >
> >> > > > > > >> >> > >
> >> > > > > > >> >>
> >> > > > > > >> >
> >> > > > > >
> >> > > > > >
> >> > > > > > --
> >> > > > > > Zhenya Stanilovsky
> >> > > > > >
> >> > > >
> >> > >
> >> >
> >>
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Denis, the issue here is that we don't know for sure. We see just one
blocking issue with accessing NioAccessObject.

And there are 3 different scenario related to this issue fix:
- fixes won't help, and we should find out other options on how to create a
direct buffer from pointer - needed for durable memory, Java 12 goes to a
later release.
- some fix would help, but other issues come, Java 12 goes to some later
release
- some from proposed fixes works, nothing else needs to be done - 1-2 days

If it latest scenario, I would include as much as we can (1-2 days extra
are comparable with minimal voting time).

BTW, I've checked scripts it does not work for me, I will ask Andrey
G/Peter for advice on Monday.

Sincerely,
Dmitriy Pavlov


пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:

> Folks,
>
> What are the efforts to support Java 12? Let's do 2.7.6 shortly if the
> fixes are time-consuming.
>
> -
> Denis
>
>
> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]>
> wrote:
>
> > Hi Igniters,
> >
> >  I would like to announce code freeze for 2.7.5. Only one open ticket is
> > there (reopened): https://issues.apache.org/jira/browse/IGNITE-11600 (if
> > we
> > can't start using Java 12 we should clearly state it in scripts/code).
> >
> > We're entering to Stabilization phase for
> > https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
> and
> > only blockers may be included into scope. See
> > https://cwiki.apache.org/confluence/display/IGNITE/Release+Process for
> > more
> > details.
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
> >
> > > Hi Denis,
> > >
> > > I'm not talking about months. In this discussion, Andrey and Ivan
> > proposed
> > > a couple of fixes that may help.
> > >
> > > It will require a day or two to check if it helps. If it not helpful
> then
> > > we should modify startup scripts to say clearly that Java 12 is not
> > > supported.
> > >
> > > Now under Java 12 Ignite suggests to set startup parameters, but even
> > with
> > > correct parameters, it fails and says please set parameters. Totally
> > > unclear for end-user.
> > >
> > > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
> > >
> > > Sincerely,
> > > Dmitriy Pavlov
> > >
> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
> > >
> > >> If the failure handler improvements will lower down a number of
> > >> false-positive cluster shutdowns then I'm for the fix inclusion to the
> > >> release.
> > >>
> > >> As for Java 12, I would put to the next release that we can make
> shortly
> > >> after this one. We don't need to wait for months if there are some
> > urgent
> > >> fixes.
> > >>
> > >> -
> > >> Denis
> > >>
> > >>
> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]>
> > >> wrote:
> > >>
> > >> > Denis, I also +1 for python fix. Today is the planned date of code
> > >> freeze,
> > >> > so today or tomorrow I'll announce it.
> > >> >
> > >> > Only one thing was suggested is the fix for failure handlers. I
> don't
> > >> > deeply understand the pros and cons, but I'm sure we should
> courteous
> > >> > enough for wait experienced folks to come to a consensus.
> > >> >
> > >> > One more thing I suggest to check after code freeze, it is Java 12.
> It
> > >> does
> > >> > not work now, but with thanks to Andrey G and Ivan P. we probably
> have
> > >> not
> > >> > very complex solution for this.
> > >> >
> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
> > >> >
> > >> > Sincerely,
> > >> > Dmitriy Pavlov
> > >> >
> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
> > >> >
> > >> > > Yes, the Python example's fix makes sense - that's a usability
> issue
> > >> and
> > >> > > doesn't affect the core components.
> > >> > >
> > >> > > In general, I would suggest us to do a code freeze and push other
> > >> changes
> > >> > > to another release. We can't include everything.
> > >> > >
> > >> > > -
> > >> > > Denis
> > >> > >
> > >> > >
> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> > >> > [hidden email]
> > >> > > >
> > >> > > wrote:
> > >> > >
> > >> > > > Hello!
> > >> > > >
> > >> > > > How about adding
> > https://issues.apache.org/jira/browse/IGNITE-11366
> > >> > > which
> > >> > > > fixes Python's release directory?
> > >> > > >
> > >> > > > I have stumbled on this issue personally.
> > >> > > >
> > >> > > > Regards,
> > >> > > > --
> > >> > > > Ilya Kasnacheev
> > >> > > >
> > >> > > >
> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
> > <[hidden email]
> > >> >:
> > >> > > >
> > >> > > > > Dmitriy, the probability of having nodes down with these bugs
> > >> under
> > >> > > heavy
> > >> > > > > load conditions is pretty high. Good candidate for
> > cherry-picking.
> > >> > > > >
> > >> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
> > >> > > > > -- Roman
> > >> > > > >
> > >> > > > >
> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy
> > >> Pavlov
> > >> > <
> > >> > > > > [hidden email]> wrote:
> > >> > > > >
> > >> > > > >  Hi,
> > >> > > > >
> > >> > > > > I've cherry-picked this commit. It seems it is critical
> because
> > it
> > >> > also
> > >> > > > > fixes storage corruption.
> > >> > > > >
> > >> > > > > Sincerely,
> > >> > > > > Dmitriy Pavlov
> > >> > > > >
> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> > >> > > > <[hidden email]
> > >> > > > > >:
> > >> > > > >
> > >> > > > > > I suppose this ticket [1] : is very useful too.
> > >> > > > > >
> > >> > > > > >
> > >> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> > >> > > > > > CorruptedTreeException during simultaneous cache put
> > operations
> > >> ]
> > >> > > > > >
> > >> > > > > > >
> > >> > > > > > >
> > >> > > > > > >------- Forwarded message -------
> > >> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
> > >> > > > > > >To: dev < [hidden email] >
> > >> > > > > > >Cc:
> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > >> > > > > > >
> > >> > > > > > >Hello Ilya,
> > >> > > > > > >
> > >> > > > > > >I do not see any issues with the mentioned test. I see the
> > >> > following
> > >> > > > > > output
> > >> > > > > > >in the logs:
> > >> > > > > > >
> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> > >> > ][main][root]
> > >> > > > >>>
> > >> > > > > > >Stopping test:
> > >> > > > > >
> > >> > > > > >
> > >> > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > >> > > > > > >in 37768 ms <<<
> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > >> > ][main][root]
> > >> > > > >>>
> > >> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest <<<
> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > >> > ][main][root]
> > >> > > > >>>
> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
> > >> > > > > > >
> > >> > > > > > >The issue with Windows may be long connection timeouts, in
> > this
> > >> > case
> > >> > > > we
> > >> > > > > > >should either split the suite into multiple ones or
> decrease
> > >> the
> > >> > SPI
> > >> > > > > > >timeouts.
> > >> > > > > > >
> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > >> > > > > [hidden email]
> > >> > > > > > >:
> > >> > > > > > >
> > >> > > > > > >> Hello!
> > >> > > > > > >>
> > >> > > > > > >> It seems that I can no longer test this case, on account
> of
> > >> > > > > > >>
> > >> > > > > > >>
> > >> > > > > >
> > >> > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > >> > > > > > >> hanging every time under Java 11 on Windows.
> > >> > > > > > >>
> > >> > > > > > >> Alexey, Ivan, can you please take a look?
> > >> > > > > > >>
> > >> > > > > > >>
> > >> > > > > > >>
> > >> > > > > >
> > >> > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > > > > > >>
> > >> > > > > > >> Regards,
> > >> > > > > > >>
> > >> > > > > > >> --
> > >> > > > > > >> Ilya Kasnacheev
> > >> > > > > > >>
> > >> > > > > > >>
> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > >> > > > > > [hidden email] >:
> > >> > > > > > >>
> > >> > > > > > >> > Hello!
> > >> > > > > > >> >
> > >> > > > > > >> > Basically there is a test that explicitly highlights
> this
> > >> > > problem,
> > >> > > > > > that
> > >> > > > > > >> is
> > >> > > > > > >> > running SSL tests on Windows + Java 11. They will hang
> on
> > >> > Master
> > >> > > > but
> > >> > > > > > >> pass
> > >> > > > > > >> > with this patch.
> > >> > > > > > >> >
> > >> > > > > > >> > I have started that on TC, results will probably be
> > >> available
> > >> > > > later
> > >> > > > > > >> today:
> > >> > > > > > >> >
> > >> > > > > > >> >
> > >> > > > > > >>
> > >> > > > > >
> > >> > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > > > > > >> > (mind the Java version).
> > >> > > > > > >> >
> > >> > > > > > >> > Regards,
> > >> > > > > > >> > --
> > >> > > > > > >> > Ilya Kasnacheev
> > >> > > > > > >> >
> > >> > > > > > >> >
> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> > >> > > [hidden email]
> > >> > > > > >:
> > >> > > > > > >> >
> > >> > > > > > >> >> Dmitry, Ilya,
> > >> > > > > > >> >>
> > >> > > > > > >> >> Yes, I've looked through those changes [1] as they can
> > >> affect
> > >> > > my
> > >> > > > > > local
> > >> > > > > > >> >> PR.  Basically, changes look good to me.
> > >> > > > > > >> >>
> > >> > > > > > >> >> I'm not an expert with CommunicationSpi component, so
> > can
> > >> > miss
> > >> > > > some
> > >> > > > > > >> >> details and I haven't tested these changes under Java
> > 11.
> > >> One
> > >> > > > more
> > >> > > > > > >> >> thing I'd like to say, I would add additional tests to
> > PR
> > >> > that
> > >> > > > will
> > >> > > > > > >> >> explicitly highlight the problem being solved.
> > >> > > > > > >> >>
> > >> > > > > > >> >>
> > >> > > > > > >> >> [1]
> https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > > > > > >> >>
> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> > >> > > > [hidden email]
> > >> > > > > >
> > >> > > > > > >> wrote:
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > Hi Igniters,
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > fix
> > https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > > Avoid
> > >> > > > > busy
> > >> > > > > > >> wait
> > >> > > > > > >> >> on
> > >> > > > > > >> >> > processWrite during SSL handshake.
> > >> > > > > > >> >> > seems to be blocker cause it is related to Java 11
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim
> > M.were
> > >> > these
> > >> > > > > > >> comments
> > >> > > > > > >> >> > addressed?
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer needed.
> > >> Changes
> > >> > > > > located
> > >> > > > > > >> in
> > >> > > > > > >> >> > GridNioServer.
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > Sincerely,
> > >> > > > > > >> >> > Dmitriy Pavlov
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
> > >> > > > > > >> >> >  https://issues.apache.org/jira/browse/IGNITE-11600
> > >> > > > > > >> >> >
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> > >> > > [hidden email]
> > >> > > > > >:
> > >> > > > > > >> >> >
> > >> > > > > > >> >> > > Huge +1
> > >> > > > > > >> >> > >
> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to our
> > >> Teamcity.
> > >> > > > > > >> >> > >
> > >> > > > > > >> >> > >
> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > >> > > > > [hidden email]
> > >> > > > > > >
> > >> > > > > > >> >> wrote:
> > >> > > > > > >> >> > > >
> > >> > > > > > >> >> > > > Hi Igniters,
> > >> > > > > > >> >> > > >
> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest at
> > >> least
> > >> > > test
> > >> > > > > our
> > >> > > > > > >> new
> > >> > > > > > >> >> tests
> > >> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > >> > > > > > >> >> > > >
> > >> > > > > > >> >> > > > Sincerely,
> > >> > > > > > >> >> > > > Dmitriy Pavlov
> > >> > > > > > >> >> > > >
> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > >> > > > > > >> < [hidden email] >:
> > >> > > > > > >> >> > > >
> > >> > > > > > >> >> > > >> Hi Ignite Developers,
> > >> > > > > > >> >> > > >>
> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a log
> with
> > >> all
> > >> > > > > commits.
> > >> > > > > > >> >> > > >>
> > >> > > > > > >> >> > > >> As far as I can see, nobody removed commits
> from
> > >> this
> > >> > > > sheet,
> > >> > > > > > so
> > >> > > > > > >> the
> > >> > > > > > >> >> > > scope
> > >> > > > > > >> >> > > >> of release will be discussed in another way:
> only
> > >> > > > explicitly
> > >> > > > > > >> >> declared
> > >> > > > > > >> >> > > >> commits will be cherry-picked.
> > >> > > > > > >> >> > > >>
> > >> > > > > > >> >> > > >> Sincerely,
> > >> > > > > > >> >> > > >> Dmitriy Pavlov
> > >> > > > > > >> >> > > >>
> > >> > > > > > >> >> > >
> > >> > > > > > >> >> > >
> > >> > > > > > >> >>
> > >> > > > > > >> >
> > >> > > > > >
> > >> > > > > >
> > >> > > > > > --
> > >> > > > > > Zhenya Stanilovsky
> > >> > > > > >
> > >> > > >
> > >> > >
> > >> >
> > >>
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Ivan P., it seems the netty approach you've proposed works well. Thank you.

Igniters, please take a look at following fix:
https://github.com/apache/ignite/pull/6384
It allows us to start under Java 12 and under Java 11- (as it).

Sincerely,
Dmitriy Pavlov

пт, 29 мар. 2019 г. в 22:57, Dmitriy Pavlov <[hidden email]>:

> Denis, the issue here is that we don't know for sure. We see just one
> blocking issue with accessing NioAccessObject.
>
> And there are 3 different scenario related to this issue fix:
> - fixes won't help, and we should find out other options on how to create
> a direct buffer from pointer - needed for durable memory, Java 12 goes to a
> later release.
> - some fix would help, but other issues come, Java 12 goes to some later
> release
> - some from proposed fixes works, nothing else needs to be done - 1-2 days
>
> If it latest scenario, I would include as much as we can (1-2 days extra
> are comparable with minimal voting time).
>
> BTW, I've checked scripts it does not work for me, I will ask Andrey
> G/Peter for advice on Monday.
>
> Sincerely,
> Dmitriy Pavlov
>
>
> пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:
>
>> Folks,
>>
>> What are the efforts to support Java 12? Let's do 2.7.6 shortly if the
>> fixes are time-consuming.
>>
>> -
>> Denis
>>
>>
>> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]>
>> wrote:
>>
>> > Hi Igniters,
>> >
>> >  I would like to announce code freeze for 2.7.5. Only one open ticket is
>> > there (reopened): https://issues.apache.org/jira/browse/IGNITE-11600
>> (if
>> > we
>> > can't start using Java 12 we should clearly state it in scripts/code).
>> >
>> > We're entering to Stabilization phase for
>> > https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
>> and
>> > only blockers may be included into scope. See
>> > https://cwiki.apache.org/confluence/display/IGNITE/Release+Process for
>> > more
>> > details.
>> >
>> > Sincerely,
>> > Dmitriy Pavlov
>> >
>> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
>> >
>> > > Hi Denis,
>> > >
>> > > I'm not talking about months. In this discussion, Andrey and Ivan
>> > proposed
>> > > a couple of fixes that may help.
>> > >
>> > > It will require a day or two to check if it helps. If it not helpful
>> then
>> > > we should modify startup scripts to say clearly that Java 12 is not
>> > > supported.
>> > >
>> > > Now under Java 12 Ignite suggests to set startup parameters, but even
>> > with
>> > > correct parameters, it fails and says please set parameters. Totally
>> > > unclear for end-user.
>> > >
>> > > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
>> > >
>> > > Sincerely,
>> > > Dmitriy Pavlov
>> > >
>> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
>> > >
>> > >> If the failure handler improvements will lower down a number of
>> > >> false-positive cluster shutdowns then I'm for the fix inclusion to
>> the
>> > >> release.
>> > >>
>> > >> As for Java 12, I would put to the next release that we can make
>> shortly
>> > >> after this one. We don't need to wait for months if there are some
>> > urgent
>> > >> fixes.
>> > >>
>> > >> -
>> > >> Denis
>> > >>
>> > >>
>> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]>
>> > >> wrote:
>> > >>
>> > >> > Denis, I also +1 for python fix. Today is the planned date of code
>> > >> freeze,
>> > >> > so today or tomorrow I'll announce it.
>> > >> >
>> > >> > Only one thing was suggested is the fix for failure handlers. I
>> don't
>> > >> > deeply understand the pros and cons, but I'm sure we should
>> courteous
>> > >> > enough for wait experienced folks to come to a consensus.
>> > >> >
>> > >> > One more thing I suggest to check after code freeze, it is Java
>> 12. It
>> > >> does
>> > >> > not work now, but with thanks to Andrey G and Ivan P. we probably
>> have
>> > >> not
>> > >> > very complex solution for this.
>> > >> >
>> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
>> > >> >
>> > >> > Sincerely,
>> > >> > Dmitriy Pavlov
>> > >> >
>> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
>> > >> >
>> > >> > > Yes, the Python example's fix makes sense - that's a usability
>> issue
>> > >> and
>> > >> > > doesn't affect the core components.
>> > >> > >
>> > >> > > In general, I would suggest us to do a code freeze and push other
>> > >> changes
>> > >> > > to another release. We can't include everything.
>> > >> > >
>> > >> > > -
>> > >> > > Denis
>> > >> > >
>> > >> > >
>> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
>> > >> > [hidden email]
>> > >> > > >
>> > >> > > wrote:
>> > >> > >
>> > >> > > > Hello!
>> > >> > > >
>> > >> > > > How about adding
>> > https://issues.apache.org/jira/browse/IGNITE-11366
>> > >> > > which
>> > >> > > > fixes Python's release directory?
>> > >> > > >
>> > >> > > > I have stumbled on this issue personally.
>> > >> > > >
>> > >> > > > Regards,
>> > >> > > > --
>> > >> > > > Ilya Kasnacheev
>> > >> > > >
>> > >> > > >
>> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
>> > <[hidden email]
>> > >> >:
>> > >> > > >
>> > >> > > > > Dmitriy, the probability of having nodes down with these bugs
>> > >> under
>> > >> > > heavy
>> > >> > > > > load conditions is pretty high. Good candidate for
>> > cherry-picking.
>> > >> > > > >
>> > >> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
>> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
>> > >> > > > > -- Roman
>> > >> > > > >
>> > >> > > > >
>> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9, Dmitriy
>> > >> Pavlov
>> > >> > <
>> > >> > > > > [hidden email]> wrote:
>> > >> > > > >
>> > >> > > > >  Hi,
>> > >> > > > >
>> > >> > > > > I've cherry-picked this commit. It seems it is critical
>> because
>> > it
>> > >> > also
>> > >> > > > > fixes storage corruption.
>> > >> > > > >
>> > >> > > > > Sincerely,
>> > >> > > > > Dmitriy Pavlov
>> > >> > > > >
>> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
>> > >> > > > <[hidden email]
>> > >> > > > > >:
>> > >> > > > >
>> > >> > > > > > I suppose this ticket [1] : is very useful too.
>> > >> > > > > >
>> > >> > > > > >
>> > >> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
>> > >> > > > > > CorruptedTreeException during simultaneous cache put
>> > operations
>> > >> ]
>> > >> > > > > >
>> > >> > > > > > >
>> > >> > > > > > >
>> > >> > > > > > >------- Forwarded message -------
>> > >> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
>> > >> > > > > > >To: dev < [hidden email] >
>> > >> > > > > > >Cc:
>> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
>> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
>> > >> > > > > > >
>> > >> > > > > > >Hello Ilya,
>> > >> > > > > > >
>> > >> > > > > > >I do not see any issues with the mentioned test. I see the
>> > >> > following
>> > >> > > > > > output
>> > >> > > > > > >in the logs:
>> > >> > > > > > >
>> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
>> > >> > ][main][root]
>> > >> > > > >>>
>> > >> > > > > > >Stopping test:
>> > >> > > > > >
>> > >> > > > > >
>> > >> > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> >
>> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
>> > >> > > > > > >in 37768 ms <<<
>> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
>> > >> > ][main][root]
>> > >> > > > >>>
>> > >> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest
>> <<<
>> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
>> > >> > ][main][root]
>> > >> > > > >>>
>> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
>> > >> > > > > > >
>> > >> > > > > > >The issue with Windows may be long connection timeouts, in
>> > this
>> > >> > case
>> > >> > > > we
>> > >> > > > > > >should either split the suite into multiple ones or
>> decrease
>> > >> the
>> > >> > SPI
>> > >> > > > > > >timeouts.
>> > >> > > > > > >
>> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
>> > >> > > > > [hidden email]
>> > >> > > > > > >:
>> > >> > > > > > >
>> > >> > > > > > >> Hello!
>> > >> > > > > > >>
>> > >> > > > > > >> It seems that I can no longer test this case, on
>> account of
>> > >> > > > > > >>
>> > >> > > > > > >>
>> > >> > > > > >
>> > >> > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> >
>> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
>> > >> > > > > > >> hanging every time under Java 11 on Windows.
>> > >> > > > > > >>
>> > >> > > > > > >> Alexey, Ivan, can you please take a look?
>> > >> > > > > > >>
>> > >> > > > > > >>
>> > >> > > > > > >>
>> > >> > > > > >
>> > >> > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > >> > > > > > >>
>> > >> > > > > > >> Regards,
>> > >> > > > > > >>
>> > >> > > > > > >> --
>> > >> > > > > > >> Ilya Kasnacheev
>> > >> > > > > > >>
>> > >> > > > > > >>
>> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
>> > >> > > > > > [hidden email] >:
>> > >> > > > > > >>
>> > >> > > > > > >> > Hello!
>> > >> > > > > > >> >
>> > >> > > > > > >> > Basically there is a test that explicitly highlights
>> this
>> > >> > > problem,
>> > >> > > > > > that
>> > >> > > > > > >> is
>> > >> > > > > > >> > running SSL tests on Windows + Java 11. They will
>> hang on
>> > >> > Master
>> > >> > > > but
>> > >> > > > > > >> pass
>> > >> > > > > > >> > with this patch.
>> > >> > > > > > >> >
>> > >> > > > > > >> > I have started that on TC, results will probably be
>> > >> available
>> > >> > > > later
>> > >> > > > > > >> today:
>> > >> > > > > > >> >
>> > >> > > > > > >> >
>> > >> > > > > > >>
>> > >> > > > > >
>> > >> > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > >> > > > > > >> > (mind the Java version).
>> > >> > > > > > >> >
>> > >> > > > > > >> > Regards,
>> > >> > > > > > >> > --
>> > >> > > > > > >> > Ilya Kasnacheev
>> > >> > > > > > >> >
>> > >> > > > > > >> >
>> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
>> > >> > > [hidden email]
>> > >> > > > > >:
>> > >> > > > > > >> >
>> > >> > > > > > >> >> Dmitry, Ilya,
>> > >> > > > > > >> >>
>> > >> > > > > > >> >> Yes, I've looked through those changes [1] as they
>> can
>> > >> affect
>> > >> > > my
>> > >> > > > > > local
>> > >> > > > > > >> >> PR.  Basically, changes look good to me.
>> > >> > > > > > >> >>
>> > >> > > > > > >> >> I'm not an expert with CommunicationSpi component, so
>> > can
>> > >> > miss
>> > >> > > > some
>> > >> > > > > > >> >> details and I haven't tested these changes under Java
>> > 11.
>> > >> One
>> > >> > > > more
>> > >> > > > > > >> >> thing I'd like to say, I would add additional tests
>> to
>> > PR
>> > >> > that
>> > >> > > > will
>> > >> > > > > > >> >> explicitly highlight the problem being solved.
>> > >> > > > > > >> >>
>> > >> > > > > > >> >>
>> > >> > > > > > >> >> [1]
>> https://issues.apache.org/jira/browse/IGNITE-11299
>> > >> > > > > > >> >>
>> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
>> > >> > > > [hidden email]
>> > >> > > > > >
>> > >> > > > > > >> wrote:
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > Hi Igniters,
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > fix
>> > https://issues.apache.org/jira/browse/IGNITE-11299
>> > >> > > Avoid
>> > >> > > > > busy
>> > >> > > > > > >> wait
>> > >> > > > > > >> >> on
>> > >> > > > > > >> >> > processWrite during SSL handshake.
>> > >> > > > > > >> >> > seems to be blocker cause it is related to Java 11
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim
>> > M.were
>> > >> > these
>> > >> > > > > > >> comments
>> > >> > > > > > >> >> > addressed?
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer needed.
>> > >> Changes
>> > >> > > > > located
>> > >> > > > > > >> in
>> > >> > > > > > >> >> > GridNioServer.
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > Sincerely,
>> > >> > > > > > >> >> > Dmitriy Pavlov
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as well:
>> > >> > > > > > >> >> >
>> https://issues.apache.org/jira/browse/IGNITE-11600
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
>> > >> > > [hidden email]
>> > >> > > > > >:
>> > >> > > > > > >> >> >
>> > >> > > > > > >> >> > > Huge +1
>> > >> > > > > > >> >> > >
>> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to our
>> > >> Teamcity.
>> > >> > > > > > >> >> > >
>> > >> > > > > > >> >> > >
>> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
>> > >> > > > > [hidden email]
>> > >> > > > > > >
>> > >> > > > > > >> >> wrote:
>> > >> > > > > > >> >> > > >
>> > >> > > > > > >> >> > > > Hi Igniters,
>> > >> > > > > > >> >> > > >
>> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest
>> at
>> > >> least
>> > >> > > test
>> > >> > > > > our
>> > >> > > > > > >> new
>> > >> > > > > > >> >> tests
>> > >> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
>> > >> > > > > > >> >> > > >
>> > >> > > > > > >> >> > > > Sincerely,
>> > >> > > > > > >> >> > > > Dmitriy Pavlov
>> > >> > > > > > >> >> > > >
>> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
>> > >> > > > > > >> < [hidden email] >:
>> > >> > > > > > >> >> > > >
>> > >> > > > > > >> >> > > >> Hi Ignite Developers,
>> > >> > > > > > >> >> > > >>
>> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a log
>> with
>> > >> all
>> > >> > > > > commits.
>> > >> > > > > > >> >> > > >>
>> > >> > > > > > >> >> > > >> As far as I can see, nobody removed commits
>> from
>> > >> this
>> > >> > > > sheet,
>> > >> > > > > > so
>> > >> > > > > > >> the
>> > >> > > > > > >> >> > > scope
>> > >> > > > > > >> >> > > >> of release will be discussed in another way:
>> only
>> > >> > > > explicitly
>> > >> > > > > > >> >> declared
>> > >> > > > > > >> >> > > >> commits will be cherry-picked.
>> > >> > > > > > >> >> > > >>
>> > >> > > > > > >> >> > > >> Sincerely,
>> > >> > > > > > >> >> > > >> Dmitriy Pavlov
>> > >> > > > > > >> >> > > >>
>> > >> > > > > > >> >> > >
>> > >> > > > > > >> >> > >
>> > >> > > > > > >> >>
>> > >> > > > > > >> >
>> > >> > > > > >
>> > >> > > > > >
>> > >> > > > > > --
>> > >> > > > > > Zhenya Stanilovsky
>> > >> > > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> > >
>> >
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Nikolay Izhikov-2
Hello, Dmitriy.

Any news about release?
Do you need assistance with it?

вт, 2 апр. 2019 г. в 20:04, Dmitriy Pavlov <[hidden email]>:

> Ivan P., it seems the netty approach you've proposed works well. Thank you.
>
> Igniters, please take a look at following fix:
> https://github.com/apache/ignite/pull/6384
> It allows us to start under Java 12 and under Java 11- (as it).
>
> Sincerely,
> Dmitriy Pavlov
>
> пт, 29 мар. 2019 г. в 22:57, Dmitriy Pavlov <[hidden email]>:
>
> > Denis, the issue here is that we don't know for sure. We see just one
> > blocking issue with accessing NioAccessObject.
> >
> > And there are 3 different scenario related to this issue fix:
> > - fixes won't help, and we should find out other options on how to create
> > a direct buffer from pointer - needed for durable memory, Java 12 goes
> to a
> > later release.
> > - some fix would help, but other issues come, Java 12 goes to some later
> > release
> > - some from proposed fixes works, nothing else needs to be done - 1-2
> days
> >
> > If it latest scenario, I would include as much as we can (1-2 days extra
> > are comparable with minimal voting time).
> >
> > BTW, I've checked scripts it does not work for me, I will ask Andrey
> > G/Peter for advice on Monday.
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> >
> > пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:
> >
> >> Folks,
> >>
> >> What are the efforts to support Java 12? Let's do 2.7.6 shortly if the
> >> fixes are time-consuming.
> >>
> >> -
> >> Denis
> >>
> >>
> >> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]>
> >> wrote:
> >>
> >> > Hi Igniters,
> >> >
> >> >  I would like to announce code freeze for 2.7.5. Only one open ticket
> is
> >> > there (reopened): https://issues.apache.org/jira/browse/IGNITE-11600
> >> (if
> >> > we
> >> > can't start using Java 12 we should clearly state it in scripts/code).
> >> >
> >> > We're entering to Stabilization phase for
> >> >
> https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
> >> and
> >> > only blockers may be included into scope. See
> >> > https://cwiki.apache.org/confluence/display/IGNITE/Release+Process
> for
> >> > more
> >> > details.
> >> >
> >> > Sincerely,
> >> > Dmitriy Pavlov
> >> >
> >> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
> >> >
> >> > > Hi Denis,
> >> > >
> >> > > I'm not talking about months. In this discussion, Andrey and Ivan
> >> > proposed
> >> > > a couple of fixes that may help.
> >> > >
> >> > > It will require a day or two to check if it helps. If it not helpful
> >> then
> >> > > we should modify startup scripts to say clearly that Java 12 is not
> >> > > supported.
> >> > >
> >> > > Now under Java 12 Ignite suggests to set startup parameters, but
> even
> >> > with
> >> > > correct parameters, it fails and says please set parameters. Totally
> >> > > unclear for end-user.
> >> > >
> >> > > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
> >> > >
> >> > > Sincerely,
> >> > > Dmitriy Pavlov
> >> > >
> >> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
> >> > >
> >> > >> If the failure handler improvements will lower down a number of
> >> > >> false-positive cluster shutdowns then I'm for the fix inclusion to
> >> the
> >> > >> release.
> >> > >>
> >> > >> As for Java 12, I would put to the next release that we can make
> >> shortly
> >> > >> after this one. We don't need to wait for months if there are some
> >> > urgent
> >> > >> fixes.
> >> > >>
> >> > >> -
> >> > >> Denis
> >> > >>
> >> > >>
> >> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <[hidden email]
> >
> >> > >> wrote:
> >> > >>
> >> > >> > Denis, I also +1 for python fix. Today is the planned date of
> code
> >> > >> freeze,
> >> > >> > so today or tomorrow I'll announce it.
> >> > >> >
> >> > >> > Only one thing was suggested is the fix for failure handlers. I
> >> don't
> >> > >> > deeply understand the pros and cons, but I'm sure we should
> >> courteous
> >> > >> > enough for wait experienced folks to come to a consensus.
> >> > >> >
> >> > >> > One more thing I suggest to check after code freeze, it is Java
> >> 12. It
> >> > >> does
> >> > >> > not work now, but with thanks to Andrey G and Ivan P. we probably
> >> have
> >> > >> not
> >> > >> > very complex solution for this.
> >> > >> >
> >> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
> >> > >> >
> >> > >> > Sincerely,
> >> > >> > Dmitriy Pavlov
> >> > >> >
> >> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
> >> > >> >
> >> > >> > > Yes, the Python example's fix makes sense - that's a usability
> >> issue
> >> > >> and
> >> > >> > > doesn't affect the core components.
> >> > >> > >
> >> > >> > > In general, I would suggest us to do a code freeze and push
> other
> >> > >> changes
> >> > >> > > to another release. We can't include everything.
> >> > >> > >
> >> > >> > > -
> >> > >> > > Denis
> >> > >> > >
> >> > >> > >
> >> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> >> > >> > [hidden email]
> >> > >> > > >
> >> > >> > > wrote:
> >> > >> > >
> >> > >> > > > Hello!
> >> > >> > > >
> >> > >> > > > How about adding
> >> > https://issues.apache.org/jira/browse/IGNITE-11366
> >> > >> > > which
> >> > >> > > > fixes Python's release directory?
> >> > >> > > >
> >> > >> > > > I have stumbled on this issue personally.
> >> > >> > > >
> >> > >> > > > Regards,
> >> > >> > > > --
> >> > >> > > > Ilya Kasnacheev
> >> > >> > > >
> >> > >> > > >
> >> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
> >> > <[hidden email]
> >> > >> >:
> >> > >> > > >
> >> > >> > > > > Dmitriy, the probability of having nodes down with these
> bugs
> >> > >> under
> >> > >> > > heavy
> >> > >> > > > > load conditions is pretty high. Good candidate for
> >> > cherry-picking.
> >> > >> > > > >
> >> > >> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> >> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
> >> > >> > > > > -- Roman
> >> > >> > > > >
> >> > >> > > > >
> >> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9,
> Dmitriy
> >> > >> Pavlov
> >> > >> > <
> >> > >> > > > > [hidden email]> wrote:
> >> > >> > > > >
> >> > >> > > > >  Hi,
> >> > >> > > > >
> >> > >> > > > > I've cherry-picked this commit. It seems it is critical
> >> because
> >> > it
> >> > >> > also
> >> > >> > > > > fixes storage corruption.
> >> > >> > > > >
> >> > >> > > > > Sincerely,
> >> > >> > > > > Dmitriy Pavlov
> >> > >> > > > >
> >> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> >> > >> > > > <[hidden email]
> >> > >> > > > > >:
> >> > >> > > > >
> >> > >> > > > > > I suppose this ticket [1] : is very useful too.
> >> > >> > > > > >
> >> > >> > > > > >
> >> > >> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873 [
> >> > >> > > > > > CorruptedTreeException during simultaneous cache put
> >> > operations
> >> > >> ]
> >> > >> > > > > >
> >> > >> > > > > > >
> >> > >> > > > > > >
> >> > >> > > > > > >------- Forwarded message -------
> >> > >> > > > > > >From: "Alexey Goncharuk" < [hidden email] >
> >> > >> > > > > > >To: dev < [hidden email] >
> >> > >> > > > > > >Cc:
> >> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
> >> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> >> > >> > > > > > >
> >> > >> > > > > > >Hello Ilya,
> >> > >> > > > > > >
> >> > >> > > > > > >I do not see any issues with the mentioned test. I see
> the
> >> > >> > following
> >> > >> > > > > > output
> >> > >> > > > > > >in the logs:
> >> > >> > > > > > >
> >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> >> > >> > ][main][root]
> >> > >> > > > >>>
> >> > >> > > > > > >Stopping test:
> >> > >> > > > > >
> >> > >> > > > > >
> >> > >> > > > >
> >> > >> > > >
> >> > >> > >
> >> > >> >
> >> > >>
> >> >
> >>
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> >> > >> > > > > > >in 37768 ms <<<
> >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> >> > >> > ][main][root]
> >> > >> > > > >>>
> >> > >> > > > > > >Stopping test class: TcpDiscoveryCoordinatorFailureTest
> >> <<<
> >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> >> > >> > ][main][root]
> >> > >> > > > >>>
> >> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
> >> > >> > > > > > >
> >> > >> > > > > > >The issue with Windows may be long connection timeouts,
> in
> >> > this
> >> > >> > case
> >> > >> > > > we
> >> > >> > > > > > >should either split the suite into multiple ones or
> >> decrease
> >> > >> the
> >> > >> > SPI
> >> > >> > > > > > >timeouts.
> >> > >> > > > > > >
> >> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> >> > >> > > > > [hidden email]
> >> > >> > > > > > >:
> >> > >> > > > > > >
> >> > >> > > > > > >> Hello!
> >> > >> > > > > > >>
> >> > >> > > > > > >> It seems that I can no longer test this case, on
> >> account of
> >> > >> > > > > > >>
> >> > >> > > > > > >>
> >> > >> > > > > >
> >> > >> > > > >
> >> > >> > > >
> >> > >> > >
> >> > >> >
> >> > >>
> >> >
> >>
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> >> > >> > > > > > >> hanging every time under Java 11 on Windows.
> >> > >> > > > > > >>
> >> > >> > > > > > >> Alexey, Ivan, can you please take a look?
> >> > >> > > > > > >>
> >> > >> > > > > > >>
> >> > >> > > > > > >>
> >> > >> > > > > >
> >> > >> > > > >
> >> > >> > > >
> >> > >> > >
> >> > >> >
> >> > >>
> >> >
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >> > >> > > > > > >>
> >> > >> > > > > > >> Regards,
> >> > >> > > > > > >>
> >> > >> > > > > > >> --
> >> > >> > > > > > >> Ilya Kasnacheev
> >> > >> > > > > > >>
> >> > >> > > > > > >>
> >> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> >> > >> > > > > > [hidden email] >:
> >> > >> > > > > > >>
> >> > >> > > > > > >> > Hello!
> >> > >> > > > > > >> >
> >> > >> > > > > > >> > Basically there is a test that explicitly highlights
> >> this
> >> > >> > > problem,
> >> > >> > > > > > that
> >> > >> > > > > > >> is
> >> > >> > > > > > >> > running SSL tests on Windows + Java 11. They will
> >> hang on
> >> > >> > Master
> >> > >> > > > but
> >> > >> > > > > > >> pass
> >> > >> > > > > > >> > with this patch.
> >> > >> > > > > > >> >
> >> > >> > > > > > >> > I have started that on TC, results will probably be
> >> > >> available
> >> > >> > > > later
> >> > >> > > > > > >> today:
> >> > >> > > > > > >> >
> >> > >> > > > > > >> >
> >> > >> > > > > > >>
> >> > >> > > > > >
> >> > >> > > > >
> >> > >> > > >
> >> > >> > >
> >> > >> >
> >> > >>
> >> >
> >>
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> >> > >> > > > > > >> > (mind the Java version).
> >> > >> > > > > > >> >
> >> > >> > > > > > >> > Regards,
> >> > >> > > > > > >> > --
> >> > >> > > > > > >> > Ilya Kasnacheev
> >> > >> > > > > > >> >
> >> > >> > > > > > >> >
> >> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> >> > >> > > [hidden email]
> >> > >> > > > > >:
> >> > >> > > > > > >> >
> >> > >> > > > > > >> >> Dmitry, Ilya,
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >> Yes, I've looked through those changes [1] as they
> >> can
> >> > >> affect
> >> > >> > > my
> >> > >> > > > > > local
> >> > >> > > > > > >> >> PR.  Basically, changes look good to me.
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >> I'm not an expert with CommunicationSpi component,
> so
> >> > can
> >> > >> > miss
> >> > >> > > > some
> >> > >> > > > > > >> >> details and I haven't tested these changes under
> Java
> >> > 11.
> >> > >> One
> >> > >> > > > more
> >> > >> > > > > > >> >> thing I'd like to say, I would add additional tests
> >> to
> >> > PR
> >> > >> > that
> >> > >> > > > will
> >> > >> > > > > > >> >> explicitly highlight the problem being solved.
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >> [1]
> >> https://issues.apache.org/jira/browse/IGNITE-11299
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> >> > >> > > > [hidden email]
> >> > >> > > > > >
> >> > >> > > > > > >> wrote:
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > Hi Igniters,
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > fix
> >> > https://issues.apache.org/jira/browse/IGNITE-11299
> >> > >> > > Avoid
> >> > >> > > > > busy
> >> > >> > > > > > >> wait
> >> > >> > > > > > >> >> on
> >> > >> > > > > > >> >> > processWrite during SSL handshake.
> >> > >> > > > > > >> >> > seems to be blocker cause it is related to Java
> 11
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K., Maxim
> >> > M.were
> >> > >> > these
> >> > >> > > > > > >> comments
> >> > >> > > > > > >> >> > addressed?
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer
> needed.
> >> > >> Changes
> >> > >> > > > > located
> >> > >> > > > > > >> in
> >> > >> > > > > > >> >> > GridNioServer.
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > Sincerely,
> >> > >> > > > > > >> >> > Dmitriy Pavlov
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as
> well:
> >> > >> > > > > > >> >> >
> >> https://issues.apache.org/jira/browse/IGNITE-11600
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> >> > >> > > [hidden email]
> >> > >> > > > > >:
> >> > >> > > > > > >> >> >
> >> > >> > > > > > >> >> > > Huge +1
> >> > >> > > > > > >> >> > >
> >> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to our
> >> > >> Teamcity.
> >> > >> > > > > > >> >> > >
> >> > >> > > > > > >> >> > >
> >> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> >> > >> > > > > [hidden email]
> >> > >> > > > > > >
> >> > >> > > > > > >> >> wrote:
> >> > >> > > > > > >> >> > > >
> >> > >> > > > > > >> >> > > > Hi Igniters,
> >> > >> > > > > > >> >> > > >
> >> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I suggest
> >> at
> >> > >> least
> >> > >> > > test
> >> > >> > > > > our
> >> > >> > > > > > >> new
> >> > >> > > > > > >> >> tests
> >> > >> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> >> > >> > > > > > >> >> > > >
> >> > >> > > > > > >> >> > > > Sincerely,
> >> > >> > > > > > >> >> > > > Dmitriy Pavlov
> >> > >> > > > > > >> >> > > >
> >> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> >> > >> > > > > > >> < [hidden email] >:
> >> > >> > > > > > >> >> > > >
> >> > >> > > > > > >> >> > > >> Hi Ignite Developers,
> >> > >> > > > > > >> >> > > >>
> >> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a log
> >> with
> >> > >> all
> >> > >> > > > > commits.
> >> > >> > > > > > >> >> > > >>
> >> > >> > > > > > >> >> > > >> As far as I can see, nobody removed commits
> >> from
> >> > >> this
> >> > >> > > > sheet,
> >> > >> > > > > > so
> >> > >> > > > > > >> the
> >> > >> > > > > > >> >> > > scope
> >> > >> > > > > > >> >> > > >> of release will be discussed in another way:
> >> only
> >> > >> > > > explicitly
> >> > >> > > > > > >> >> declared
> >> > >> > > > > > >> >> > > >> commits will be cherry-picked.
> >> > >> > > > > > >> >> > > >>
> >> > >> > > > > > >> >> > > >> Sincerely,
> >> > >> > > > > > >> >> > > >> Dmitriy Pavlov
> >> > >> > > > > > >> >> > > >>
> >> > >> > > > > > >> >> > >
> >> > >> > > > > > >> >> > >
> >> > >> > > > > > >> >>
> >> > >> > > > > > >> >
> >> > >> > > > > >
> >> > >> > > > > >
> >> > >> > > > > > --
> >> > >> > > > > > Zhenya Stanilovsky
> >> > >> > > > > >
> >> > >> > > >
> >> > >> > >
> >> > >> >
> >> > >>
> >> > >
> >> >
> >>
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Hi, Nikolay,

thanks for offering help.

Performance testing of new fix seems to be almost done.

After merging of the last ticket (I hope it will be tomorrow) I will
continue with RC building. There can be some issues related to scripts.

Sincerely,
Dmitriy Pavlov

вт, 9 апр. 2019 г. в 09:53, Nikolay Izhikov <[hidden email]>:

> Hello, Dmitriy.
>
> Any news about release?
> Do you need assistance with it?
>
> вт, 2 апр. 2019 г. в 20:04, Dmitriy Pavlov <[hidden email]>:
>
> > Ivan P., it seems the netty approach you've proposed works well. Thank
> you.
> >
> > Igniters, please take a look at following fix:
> > https://github.com/apache/ignite/pull/6384
> > It allows us to start under Java 12 and under Java 11- (as it).
> >
> > Sincerely,
> > Dmitriy Pavlov
> >
> > пт, 29 мар. 2019 г. в 22:57, Dmitriy Pavlov <[hidden email]>:
> >
> > > Denis, the issue here is that we don't know for sure. We see just one
> > > blocking issue with accessing NioAccessObject.
> > >
> > > And there are 3 different scenario related to this issue fix:
> > > - fixes won't help, and we should find out other options on how to
> create
> > > a direct buffer from pointer - needed for durable memory, Java 12 goes
> > to a
> > > later release.
> > > - some fix would help, but other issues come, Java 12 goes to some
> later
> > > release
> > > - some from proposed fixes works, nothing else needs to be done - 1-2
> > days
> > >
> > > If it latest scenario, I would include as much as we can (1-2 days
> extra
> > > are comparable with minimal voting time).
> > >
> > > BTW, I've checked scripts it does not work for me, I will ask Andrey
> > > G/Peter for advice on Monday.
> > >
> > > Sincerely,
> > > Dmitriy Pavlov
> > >
> > >
> > > пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:
> > >
> > >> Folks,
> > >>
> > >> What are the efforts to support Java 12? Let's do 2.7.6 shortly if the
> > >> fixes are time-consuming.
> > >>
> > >> -
> > >> Denis
> > >>
> > >>
> > >> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]>
> > >> wrote:
> > >>
> > >> > Hi Igniters,
> > >> >
> > >> >  I would like to announce code freeze for 2.7.5. Only one open
> ticket
> > is
> > >> > there (reopened):
> https://issues.apache.org/jira/browse/IGNITE-11600
> > >> (if
> > >> > we
> > >> > can't start using Java 12 we should clearly state it in
> scripts/code).
> > >> >
> > >> > We're entering to Stabilization phase for
> > >> >
> > https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
> > >> and
> > >> > only blockers may be included into scope. See
> > >> > https://cwiki.apache.org/confluence/display/IGNITE/Release+Process
> > for
> > >> > more
> > >> > details.
> > >> >
> > >> > Sincerely,
> > >> > Dmitriy Pavlov
> > >> >
> > >> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
> > >> >
> > >> > > Hi Denis,
> > >> > >
> > >> > > I'm not talking about months. In this discussion, Andrey and Ivan
> > >> > proposed
> > >> > > a couple of fixes that may help.
> > >> > >
> > >> > > It will require a day or two to check if it helps. If it not
> helpful
> > >> then
> > >> > > we should modify startup scripts to say clearly that Java 12 is
> not
> > >> > > supported.
> > >> > >
> > >> > > Now under Java 12 Ignite suggests to set startup parameters, but
> > even
> > >> > with
> > >> > > correct parameters, it fails and says please set parameters.
> Totally
> > >> > > unclear for end-user.
> > >> > >
> > >> > > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
> > >> > >
> > >> > > Sincerely,
> > >> > > Dmitriy Pavlov
> > >> > >
> > >> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
> > >> > >
> > >> > >> If the failure handler improvements will lower down a number of
> > >> > >> false-positive cluster shutdowns then I'm for the fix inclusion
> to
> > >> the
> > >> > >> release.
> > >> > >>
> > >> > >> As for Java 12, I would put to the next release that we can make
> > >> shortly
> > >> > >> after this one. We don't need to wait for months if there are
> some
> > >> > urgent
> > >> > >> fixes.
> > >> > >>
> > >> > >> -
> > >> > >> Denis
> > >> > >>
> > >> > >>
> > >> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <
> [hidden email]
> > >
> > >> > >> wrote:
> > >> > >>
> > >> > >> > Denis, I also +1 for python fix. Today is the planned date of
> > code
> > >> > >> freeze,
> > >> > >> > so today or tomorrow I'll announce it.
> > >> > >> >
> > >> > >> > Only one thing was suggested is the fix for failure handlers. I
> > >> don't
> > >> > >> > deeply understand the pros and cons, but I'm sure we should
> > >> courteous
> > >> > >> > enough for wait experienced folks to come to a consensus.
> > >> > >> >
> > >> > >> > One more thing I suggest to check after code freeze, it is Java
> > >> 12. It
> > >> > >> does
> > >> > >> > not work now, but with thanks to Andrey G and Ivan P. we
> probably
> > >> have
> > >> > >> not
> > >> > >> > very complex solution for this.
> > >> > >> >
> > >> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
> > >> > >> >
> > >> > >> > Sincerely,
> > >> > >> > Dmitriy Pavlov
> > >> > >> >
> > >> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
> > >> > >> >
> > >> > >> > > Yes, the Python example's fix makes sense - that's a
> usability
> > >> issue
> > >> > >> and
> > >> > >> > > doesn't affect the core components.
> > >> > >> > >
> > >> > >> > > In general, I would suggest us to do a code freeze and push
> > other
> > >> > >> changes
> > >> > >> > > to another release. We can't include everything.
> > >> > >> > >
> > >> > >> > > -
> > >> > >> > > Denis
> > >> > >> > >
> > >> > >> > >
> > >> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
> > >> > >> > [hidden email]
> > >> > >> > > >
> > >> > >> > > wrote:
> > >> > >> > >
> > >> > >> > > > Hello!
> > >> > >> > > >
> > >> > >> > > > How about adding
> > >> > https://issues.apache.org/jira/browse/IGNITE-11366
> > >> > >> > > which
> > >> > >> > > > fixes Python's release directory?
> > >> > >> > > >
> > >> > >> > > > I have stumbled on this issue personally.
> > >> > >> > > >
> > >> > >> > > > Regards,
> > >> > >> > > > --
> > >> > >> > > > Ilya Kasnacheev
> > >> > >> > > >
> > >> > >> > > >
> > >> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
> > >> > <[hidden email]
> > >> > >> >:
> > >> > >> > > >
> > >> > >> > > > > Dmitriy, the probability of having nodes down with these
> > bugs
> > >> > >> under
> > >> > >> > > heavy
> > >> > >> > > > > load conditions is pretty high. Good candidate for
> > >> > cherry-picking.
> > >> > >> > > > >
> > >> > >> > > > > [1] https://issues.apache.org/jira/browse/IGNITE-9803[2]
> > >> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
> > >> > >> > > > > -- Roman
> > >> > >> > > > >
> > >> > >> > > > >
> > >> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9,
> > Dmitriy
> > >> > >> Pavlov
> > >> > >> > <
> > >> > >> > > > > [hidden email]> wrote:
> > >> > >> > > > >
> > >> > >> > > > >  Hi,
> > >> > >> > > > >
> > >> > >> > > > > I've cherry-picked this commit. It seems it is critical
> > >> because
> > >> > it
> > >> > >> > also
> > >> > >> > > > > fixes storage corruption.
> > >> > >> > > > >
> > >> > >> > > > > Sincerely,
> > >> > >> > > > > Dmitriy Pavlov
> > >> > >> > > > >
> > >> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
> > >> > >> > > > <[hidden email]
> > >> > >> > > > > >:
> > >> > >> > > > >
> > >> > >> > > > > > I suppose this ticket [1] : is very useful too.
> > >> > >> > > > > >
> > >> > >> > > > > >
> > >> > >> > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-10873
> [
> > >> > >> > > > > > CorruptedTreeException during simultaneous cache put
> > >> > operations
> > >> > >> ]
> > >> > >> > > > > >
> > >> > >> > > > > > >
> > >> > >> > > > > > >
> > >> > >> > > > > > >------- Forwarded message -------
> > >> > >> > > > > > >From: "Alexey Goncharuk" < [hidden email]
> >
> > >> > >> > > > > > >To: dev < [hidden email] >
> > >> > >> > > > > > >Cc:
> > >> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
> > >> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
> > >> > >> > > > > > >
> > >> > >> > > > > > >Hello Ilya,
> > >> > >> > > > > > >
> > >> > >> > > > > > >I do not see any issues with the mentioned test. I see
> > the
> > >> > >> > following
> > >> > >> > > > > > output
> > >> > >> > > > > > >in the logs:
> > >> > >> > > > > > >
> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,970][INFO
> > >> > >> > ][main][root]
> > >> > >> > > > >>>
> > >> > >> > > > > > >Stopping test:
> > >> > >> > > > > >
> > >> > >> > > > > >
> > >> > >> > > > >
> > >> > >> > > >
> > >> > >> > >
> > >> > >> >
> > >> > >>
> > >> >
> > >>
> >
> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
> > >> > >> > > > > > >in 37768 ms <<<
> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > >> > >> > ][main][root]
> > >> > >> > > > >>>
> > >> > >> > > > > > >Stopping test class:
> TcpDiscoveryCoordinatorFailureTest
> > >> <<<
> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22 21:41:44,971][INFO
> > >> > >> > ][main][root]
> > >> > >> > > > >>>
> > >> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
> > >> > >> > > > > > >
> > >> > >> > > > > > >The issue with Windows may be long connection
> timeouts,
> > in
> > >> > this
> > >> > >> > case
> > >> > >> > > > we
> > >> > >> > > > > > >should either split the suite into multiple ones or
> > >> decrease
> > >> > >> the
> > >> > >> > SPI
> > >> > >> > > > > > >timeouts.
> > >> > >> > > > > > >
> > >> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
> > >> > >> > > > > [hidden email]
> > >> > >> > > > > > >:
> > >> > >> > > > > > >
> > >> > >> > > > > > >> Hello!
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> It seems that I can no longer test this case, on
> > >> account of
> > >> > >> > > > > > >>
> > >> > >> > > > > > >>
> > >> > >> > > > > >
> > >> > >> > > > >
> > >> > >> > > >
> > >> > >> > >
> > >> > >> >
> > >> > >>
> > >> >
> > >>
> >
> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
> > >> > >> > > > > > >> hanging every time under Java 11 on Windows.
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> Alexey, Ivan, can you please take a look?
> > >> > >> > > > > > >>
> > >> > >> > > > > > >>
> > >> > >> > > > > > >>
> > >> > >> > > > > >
> > >> > >> > > > >
> > >> > >> > > >
> > >> > >> > >
> > >> > >> >
> > >> > >>
> > >> >
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> Regards,
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> --
> > >> > >> > > > > > >> Ilya Kasnacheev
> > >> > >> > > > > > >>
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
> > >> > >> > > > > > [hidden email] >:
> > >> > >> > > > > > >>
> > >> > >> > > > > > >> > Hello!
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> > Basically there is a test that explicitly
> highlights
> > >> this
> > >> > >> > > problem,
> > >> > >> > > > > > that
> > >> > >> > > > > > >> is
> > >> > >> > > > > > >> > running SSL tests on Windows + Java 11. They will
> > >> hang on
> > >> > >> > Master
> > >> > >> > > > but
> > >> > >> > > > > > >> pass
> > >> > >> > > > > > >> > with this patch.
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> > I have started that on TC, results will probably
> be
> > >> > >> available
> > >> > >> > > > later
> > >> > >> > > > > > >> today:
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >>
> > >> > >> > > > > >
> > >> > >> > > > >
> > >> > >> > > >
> > >> > >> > >
> > >> > >> >
> > >> > >>
> > >> >
> > >>
> >
> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
> > >> > >> > > > > > >> > (mind the Java version).
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> > Regards,
> > >> > >> > > > > > >> > --
> > >> > >> > > > > > >> > Ilya Kasnacheev
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
> > >> > >> > > [hidden email]
> > >> > >> > > > > >:
> > >> > >> > > > > > >> >
> > >> > >> > > > > > >> >> Dmitry, Ilya,
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >> Yes, I've looked through those changes [1] as
> they
> > >> can
> > >> > >> affect
> > >> > >> > > my
> > >> > >> > > > > > local
> > >> > >> > > > > > >> >> PR.  Basically, changes look good to me.
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >> I'm not an expert with CommunicationSpi
> component,
> > so
> > >> > can
> > >> > >> > miss
> > >> > >> > > > some
> > >> > >> > > > > > >> >> details and I haven't tested these changes under
> > Java
> > >> > 11.
> > >> > >> One
> > >> > >> > > > more
> > >> > >> > > > > > >> >> thing I'd like to say, I would add additional
> tests
> > >> to
> > >> > PR
> > >> > >> > that
> > >> > >> > > > will
> > >> > >> > > > > > >> >> explicitly highlight the problem being solved.
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >> [1]
> > >> https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
> > >> > >> > > > [hidden email]
> > >> > >> > > > > >
> > >> > >> > > > > > >> wrote:
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > Hi Igniters,
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > fix
> > >> > https://issues.apache.org/jira/browse/IGNITE-11299
> > >> > >> > > Avoid
> > >> > >> > > > > busy
> > >> > >> > > > > > >> wait
> > >> > >> > > > > > >> >> on
> > >> > >> > > > > > >> >> > processWrite during SSL handshake.
> > >> > >> > > > > > >> >> > seems to be blocker cause it is related to Java
> > 11
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K.,
> Maxim
> > >> > M.were
> > >> > >> > these
> > >> > >> > > > > > >> comments
> > >> > >> > > > > > >> >> > addressed?
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer
> > needed.
> > >> > >> Changes
> > >> > >> > > > > located
> > >> > >> > > > > > >> in
> > >> > >> > > > > > >> >> > GridNioServer.
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > Sincerely,
> > >> > >> > > > > > >> >> > Dmitriy Pavlov
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as
> > well:
> > >> > >> > > > > > >> >> >
> > >> https://issues.apache.org/jira/browse/IGNITE-11600
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
> > >> > >> > > [hidden email]
> > >> > >> > > > > >:
> > >> > >> > > > > > >> >> >
> > >> > >> > > > > > >> >> > > Huge +1
> > >> > >> > > > > > >> >> > >
> > >> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to
> our
> > >> > >> Teamcity.
> > >> > >> > > > > > >> >> > >
> > >> > >> > > > > > >> >> > >
> > >> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
> > >> > >> > > > > [hidden email]
> > >> > >> > > > > > >
> > >> > >> > > > > > >> >> wrote:
> > >> > >> > > > > > >> >> > > >
> > >> > >> > > > > > >> >> > > > Hi Igniters,
> > >> > >> > > > > > >> >> > > >
> > >> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I
> suggest
> > >> at
> > >> > >> least
> > >> > >> > > test
> > >> > >> > > > > our
> > >> > >> > > > > > >> new
> > >> > >> > > > > > >> >> tests
> > >> > >> > > > > > >> >> > > > scripts with a couple of Java builds. WDYT?
> > >> > >> > > > > > >> >> > > >
> > >> > >> > > > > > >> >> > > > Sincerely,
> > >> > >> > > > > > >> >> > > > Dmitriy Pavlov
> > >> > >> > > > > > >> >> > > >
> > >> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy Pavlov
> > >> > >> > > > > > >> < [hidden email] >:
> > >> > >> > > > > > >> >> > > >
> > >> > >> > > > > > >> >> > > >> Hi Ignite Developers,
> > >> > >> > > > > > >> >> > > >>
> > >> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a
> log
> > >> with
> > >> > >> all
> > >> > >> > > > > commits.
> > >> > >> > > > > > >> >> > > >>
> > >> > >> > > > > > >> >> > > >> As far as I can see, nobody removed
> commits
> > >> from
> > >> > >> this
> > >> > >> > > > sheet,
> > >> > >> > > > > > so
> > >> > >> > > > > > >> the
> > >> > >> > > > > > >> >> > > scope
> > >> > >> > > > > > >> >> > > >> of release will be discussed in another
> way:
> > >> only
> > >> > >> > > > explicitly
> > >> > >> > > > > > >> >> declared
> > >> > >> > > > > > >> >> > > >> commits will be cherry-picked.
> > >> > >> > > > > > >> >> > > >>
> > >> > >> > > > > > >> >> > > >> Sincerely,
> > >> > >> > > > > > >> >> > > >> Dmitriy Pavlov
> > >> > >> > > > > > >> >> > > >>
> > >> > >> > > > > > >> >> > >
> > >> > >> > > > > > >> >> > >
> > >> > >> > > > > > >> >>
> > >> > >> > > > > > >> >
> > >> > >> > > > > >
> > >> > >> > > > > >
> > >> > >> > > > > > --
> > >> > >> > > > > > Zhenya Stanilovsky
> > >> > >> > > > > >
> > >> > >> > > >
> > >> > >> > >
> > >> > >> >
> > >> > >>
> > >> > >
> > >> >
> > >>
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Hi Igniters,

A bit of update: Last change fixing GridUnsafe for Java 12 was merged to
master and to 2.7.5

Overnight run contains a number of failures considered as blockers by
Apache Ignite Teamcity Bot (comparing to 2.7):
- Platform .NET (Inspections) always failed- Critical F.R.: 100,0%
- Platform C++ (Linux Clang) & Platform C++ (Linux)* failed in most cases -
Critical F.R.: 60,0%
- Disk Page Compressions - Module is N/A in 2.7.5
- [Inspections] Core - always failed
- Hibernate 5.3 - Module is N/A in 2.7.5
- Platform C++ (Win x64 | Release) - also failed in 2.7

And 2 more tests failure considered as blockers (now it is in re-run
state). If re-run shows these tests are passing I proceed with RC
preparation.

Sincerely,
Dmitriy Pavlov

вт, 9 апр. 2019 г. в 10:03, Dmitriy Pavlov <[hidden email]>:

> Hi, Nikolay,
>
> thanks for offering help.
>
> Performance testing of new fix seems to be almost done.
>
> After merging of the last ticket (I hope it will be tomorrow) I will
> continue with RC building. There can be some issues related to scripts.
>
> Sincerely,
> Dmitriy Pavlov
>
> вт, 9 апр. 2019 г. в 09:53, Nikolay Izhikov <[hidden email]>:
>
>> Hello, Dmitriy.
>>
>> Any news about release?
>> Do you need assistance with it?
>>
>> вт, 2 апр. 2019 г. в 20:04, Dmitriy Pavlov <[hidden email]>:
>>
>> > Ivan P., it seems the netty approach you've proposed works well. Thank
>> you.
>> >
>> > Igniters, please take a look at following fix:
>> > https://github.com/apache/ignite/pull/6384
>> > It allows us to start under Java 12 and under Java 11- (as it).
>> >
>> > Sincerely,
>> > Dmitriy Pavlov
>> >
>> > пт, 29 мар. 2019 г. в 22:57, Dmitriy Pavlov <[hidden email]>:
>> >
>> > > Denis, the issue here is that we don't know for sure. We see just one
>> > > blocking issue with accessing NioAccessObject.
>> > >
>> > > And there are 3 different scenario related to this issue fix:
>> > > - fixes won't help, and we should find out other options on how to
>> create
>> > > a direct buffer from pointer - needed for durable memory, Java 12 goes
>> > to a
>> > > later release.
>> > > - some fix would help, but other issues come, Java 12 goes to some
>> later
>> > > release
>> > > - some from proposed fixes works, nothing else needs to be done - 1-2
>> > days
>> > >
>> > > If it latest scenario, I would include as much as we can (1-2 days
>> extra
>> > > are comparable with minimal voting time).
>> > >
>> > > BTW, I've checked scripts it does not work for me, I will ask Andrey
>> > > G/Peter for advice on Monday.
>> > >
>> > > Sincerely,
>> > > Dmitriy Pavlov
>> > >
>> > >
>> > > пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:
>> > >
>> > >> Folks,
>> > >>
>> > >> What are the efforts to support Java 12? Let's do 2.7.6 shortly if
>> the
>> > >> fixes are time-consuming.
>> > >>
>> > >> -
>> > >> Denis
>> > >>
>> > >>
>> > >> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]>
>> > >> wrote:
>> > >>
>> > >> > Hi Igniters,
>> > >> >
>> > >> >  I would like to announce code freeze for 2.7.5. Only one open
>> ticket
>> > is
>> > >> > there (reopened):
>> https://issues.apache.org/jira/browse/IGNITE-11600
>> > >> (if
>> > >> > we
>> > >> > can't start using Java 12 we should clearly state it in
>> scripts/code).
>> > >> >
>> > >> > We're entering to Stabilization phase for
>> > >> >
>> > https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
>> > >> and
>> > >> > only blockers may be included into scope. See
>> > >> > https://cwiki.apache.org/confluence/display/IGNITE/Release+Process
>> > for
>> > >> > more
>> > >> > details.
>> > >> >
>> > >> > Sincerely,
>> > >> > Dmitriy Pavlov
>> > >> >
>> > >> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
>> > >> >
>> > >> > > Hi Denis,
>> > >> > >
>> > >> > > I'm not talking about months. In this discussion, Andrey and Ivan
>> > >> > proposed
>> > >> > > a couple of fixes that may help.
>> > >> > >
>> > >> > > It will require a day or two to check if it helps. If it not
>> helpful
>> > >> then
>> > >> > > we should modify startup scripts to say clearly that Java 12 is
>> not
>> > >> > > supported.
>> > >> > >
>> > >> > > Now under Java 12 Ignite suggests to set startup parameters, but
>> > even
>> > >> > with
>> > >> > > correct parameters, it fails and says please set parameters.
>> Totally
>> > >> > > unclear for end-user.
>> > >> > >
>> > >> > > I've reopened https://issues.apache.org/jira/browse/IGNITE-11600
>> > >> > >
>> > >> > > Sincerely,
>> > >> > > Dmitriy Pavlov
>> > >> > >
>> > >> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
>> > >> > >
>> > >> > >> If the failure handler improvements will lower down a number of
>> > >> > >> false-positive cluster shutdowns then I'm for the fix inclusion
>> to
>> > >> the
>> > >> > >> release.
>> > >> > >>
>> > >> > >> As for Java 12, I would put to the next release that we can make
>> > >> shortly
>> > >> > >> after this one. We don't need to wait for months if there are
>> some
>> > >> > urgent
>> > >> > >> fixes.
>> > >> > >>
>> > >> > >> -
>> > >> > >> Denis
>> > >> > >>
>> > >> > >>
>> > >> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <
>> [hidden email]
>> > >
>> > >> > >> wrote:
>> > >> > >>
>> > >> > >> > Denis, I also +1 for python fix. Today is the planned date of
>> > code
>> > >> > >> freeze,
>> > >> > >> > so today or tomorrow I'll announce it.
>> > >> > >> >
>> > >> > >> > Only one thing was suggested is the fix for failure handlers.
>> I
>> > >> don't
>> > >> > >> > deeply understand the pros and cons, but I'm sure we should
>> > >> courteous
>> > >> > >> > enough for wait experienced folks to come to a consensus.
>> > >> > >> >
>> > >> > >> > One more thing I suggest to check after code freeze, it is
>> Java
>> > >> 12. It
>> > >> > >> does
>> > >> > >> > not work now, but with thanks to Andrey G and Ivan P. we
>> probably
>> > >> have
>> > >> > >> not
>> > >> > >> > very complex solution for this.
>> > >> > >> >
>> > >> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
>> > >> > >> >
>> > >> > >> > Sincerely,
>> > >> > >> > Dmitriy Pavlov
>> > >> > >> >
>> > >> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]>:
>> > >> > >> >
>> > >> > >> > > Yes, the Python example's fix makes sense - that's a
>> usability
>> > >> issue
>> > >> > >> and
>> > >> > >> > > doesn't affect the core components.
>> > >> > >> > >
>> > >> > >> > > In general, I would suggest us to do a code freeze and push
>> > other
>> > >> > >> changes
>> > >> > >> > > to another release. We can't include everything.
>> > >> > >> > >
>> > >> > >> > > -
>> > >> > >> > > Denis
>> > >> > >> > >
>> > >> > >> > >
>> > >> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
>> > >> > >> > [hidden email]
>> > >> > >> > > >
>> > >> > >> > > wrote:
>> > >> > >> > >
>> > >> > >> > > > Hello!
>> > >> > >> > > >
>> > >> > >> > > > How about adding
>> > >> > https://issues.apache.org/jira/browse/IGNITE-11366
>> > >> > >> > > which
>> > >> > >> > > > fixes Python's release directory?
>> > >> > >> > > >
>> > >> > >> > > > I have stumbled on this issue personally.
>> > >> > >> > > >
>> > >> > >> > > > Regards,
>> > >> > >> > > > --
>> > >> > >> > > > Ilya Kasnacheev
>> > >> > >> > > >
>> > >> > >> > > >
>> > >> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
>> > >> > <[hidden email]
>> > >> > >> >:
>> > >> > >> > > >
>> > >> > >> > > > > Dmitriy, the probability of having nodes down with these
>> > bugs
>> > >> > >> under
>> > >> > >> > > heavy
>> > >> > >> > > > > load conditions is pretty high. Good candidate for
>> > >> > cherry-picking.
>> > >> > >> > > > >
>> > >> > >> > > > > [1]
>> https://issues.apache.org/jira/browse/IGNITE-9803[2]
>> > >> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
>> > >> > >> > > > > -- Roman
>> > >> > >> > > > >
>> > >> > >> > > > >
>> > >> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9,
>> > Dmitriy
>> > >> > >> Pavlov
>> > >> > >> > <
>> > >> > >> > > > > [hidden email]> wrote:
>> > >> > >> > > > >
>> > >> > >> > > > >  Hi,
>> > >> > >> > > > >
>> > >> > >> > > > > I've cherry-picked this commit. It seems it is critical
>> > >> because
>> > >> > it
>> > >> > >> > also
>> > >> > >> > > > > fixes storage corruption.
>> > >> > >> > > > >
>> > >> > >> > > > > Sincerely,
>> > >> > >> > > > > Dmitriy Pavlov
>> > >> > >> > > > >
>> > >> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
>> > >> > >> > > > <[hidden email]
>> > >> > >> > > > > >:
>> > >> > >> > > > >
>> > >> > >> > > > > > I suppose this ticket [1] : is very useful too.
>> > >> > >> > > > > >
>> > >> > >> > > > > >
>> > >> > >> > > > > > [1]
>> https://issues.apache.org/jira/browse/IGNITE-10873 [
>> > >> > >> > > > > > CorruptedTreeException during simultaneous cache put
>> > >> > operations
>> > >> > >> ]
>> > >> > >> > > > > >
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >------- Forwarded message -------
>> > >> > >> > > > > > >From: "Alexey Goncharuk" <
>> [hidden email] >
>> > >> > >> > > > > > >To: dev < [hidden email] >
>> > >> > >> > > > > > >Cc:
>> > >> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
>> > >> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >Hello Ilya,
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >I do not see any issues with the mentioned test. I
>> see
>> > the
>> > >> > >> > following
>> > >> > >> > > > > > output
>> > >> > >> > > > > > >in the logs:
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>> 21:41:44,970][INFO
>> > >> > >> > ][main][root]
>> > >> > >> > > > >>>
>> > >> > >> > > > > > >Stopping test:
>> > >> > >> > > > > >
>> > >> > >> > > > > >
>> > >> > >> > > > >
>> > >> > >> > > >
>> > >> > >> > >
>> > >> > >> >
>> > >> > >>
>> > >> >
>> > >>
>> >
>> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
>> > >> > >> > > > > > >in 37768 ms <<<
>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>> 21:41:44,971][INFO
>> > >> > >> > ][main][root]
>> > >> > >> > > > >>>
>> > >> > >> > > > > > >Stopping test class:
>> TcpDiscoveryCoordinatorFailureTest
>> > >> <<<
>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>> 21:41:44,971][INFO
>> > >> > >> > ][main][root]
>> > >> > >> > > > >>>
>> > >> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >The issue with Windows may be long connection
>> timeouts,
>> > in
>> > >> > this
>> > >> > >> > case
>> > >> > >> > > > we
>> > >> > >> > > > > > >should either split the suite into multiple ones or
>> > >> decrease
>> > >> > >> the
>> > >> > >> > SPI
>> > >> > >> > > > > > >timeouts.
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
>> > >> > >> > > > > [hidden email]
>> > >> > >> > > > > > >:
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >> Hello!
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> It seems that I can no longer test this case, on
>> > >> account of
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >>
>> > >> > >> > > > > >
>> > >> > >> > > > >
>> > >> > >> > > >
>> > >> > >> > >
>> > >> > >> >
>> > >> > >>
>> > >> >
>> > >>
>> >
>> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
>> > >> > >> > > > > > >> hanging every time under Java 11 on Windows.
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> Alexey, Ivan, can you please take a look?
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >>
>> > >> > >> > > > > >
>> > >> > >> > > > >
>> > >> > >> > > >
>> > >> > >> > >
>> > >> > >> >
>> > >> > >>
>> > >> >
>> > >>
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> Regards,
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> --
>> > >> > >> > > > > > >> Ilya Kasnacheev
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
>> > >> > >> > > > > > [hidden email] >:
>> > >> > >> > > > > > >>
>> > >> > >> > > > > > >> > Hello!
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> > Basically there is a test that explicitly
>> highlights
>> > >> this
>> > >> > >> > > problem,
>> > >> > >> > > > > > that
>> > >> > >> > > > > > >> is
>> > >> > >> > > > > > >> > running SSL tests on Windows + Java 11. They will
>> > >> hang on
>> > >> > >> > Master
>> > >> > >> > > > but
>> > >> > >> > > > > > >> pass
>> > >> > >> > > > > > >> > with this patch.
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> > I have started that on TC, results will probably
>> be
>> > >> > >> available
>> > >> > >> > > > later
>> > >> > >> > > > > > >> today:
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >>
>> > >> > >> > > > > >
>> > >> > >> > > > >
>> > >> > >> > > >
>> > >> > >> > >
>> > >> > >> >
>> > >> > >>
>> > >> >
>> > >>
>> >
>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>> > >> > >> > > > > > >> > (mind the Java version).
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> > Regards,
>> > >> > >> > > > > > >> > --
>> > >> > >> > > > > > >> > Ilya Kasnacheev
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
>> > >> > >> > > [hidden email]
>> > >> > >> > > > > >:
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > > >> >> Dmitry, Ilya,
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >> Yes, I've looked through those changes [1] as
>> they
>> > >> can
>> > >> > >> affect
>> > >> > >> > > my
>> > >> > >> > > > > > local
>> > >> > >> > > > > > >> >> PR.  Basically, changes look good to me.
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >> I'm not an expert with CommunicationSpi
>> component,
>> > so
>> > >> > can
>> > >> > >> > miss
>> > >> > >> > > > some
>> > >> > >> > > > > > >> >> details and I haven't tested these changes under
>> > Java
>> > >> > 11.
>> > >> > >> One
>> > >> > >> > > > more
>> > >> > >> > > > > > >> >> thing I'd like to say, I would add additional
>> tests
>> > >> to
>> > >> > PR
>> > >> > >> > that
>> > >> > >> > > > will
>> > >> > >> > > > > > >> >> explicitly highlight the problem being solved.
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >> [1]
>> > >> https://issues.apache.org/jira/browse/IGNITE-11299
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
>> > >> > >> > > > [hidden email]
>> > >> > >> > > > > >
>> > >> > >> > > > > > >> wrote:
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > Hi Igniters,
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > fix
>> > >> > https://issues.apache.org/jira/browse/IGNITE-11299
>> > >> > >> > > Avoid
>> > >> > >> > > > > busy
>> > >> > >> > > > > > >> wait
>> > >> > >> > > > > > >> >> on
>> > >> > >> > > > > > >> >> > processWrite during SSL handshake.
>> > >> > >> > > > > > >> >> > seems to be blocker cause it is related to
>> Java
>> > 11
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K.,
>> Maxim
>> > >> > M.were
>> > >> > >> > these
>> > >> > >> > > > > > >> comments
>> > >> > >> > > > > > >> >> > addressed?
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer
>> > needed.
>> > >> > >> Changes
>> > >> > >> > > > > located
>> > >> > >> > > > > > >> in
>> > >> > >> > > > > > >> >> > GridNioServer.
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > Sincerely,
>> > >> > >> > > > > > >> >> > Dmitriy Pavlov
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as
>> > well:
>> > >> > >> > > > > > >> >> >
>> > >> https://issues.apache.org/jira/browse/IGNITE-11600
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
>> > >> > >> > > [hidden email]
>> > >> > >> > > > > >:
>> > >> > >> > > > > > >> >> >
>> > >> > >> > > > > > >> >> > > Huge +1
>> > >> > >> > > > > > >> >> > >
>> > >> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to
>> our
>> > >> > >> Teamcity.
>> > >> > >> > > > > > >> >> > >
>> > >> > >> > > > > > >> >> > >
>> > >> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov <
>> > >> > >> > > > > [hidden email]
>> > >> > >> > > > > > >
>> > >> > >> > > > > > >> >> wrote:
>> > >> > >> > > > > > >> >> > > >
>> > >> > >> > > > > > >> >> > > > Hi Igniters,
>> > >> > >> > > > > > >> >> > > >
>> > >> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I
>> suggest
>> > >> at
>> > >> > >> least
>> > >> > >> > > test
>> > >> > >> > > > > our
>> > >> > >> > > > > > >> new
>> > >> > >> > > > > > >> >> tests
>> > >> > >> > > > > > >> >> > > > scripts with a couple of Java builds.
>> WDYT?
>> > >> > >> > > > > > >> >> > > >
>> > >> > >> > > > > > >> >> > > > Sincerely,
>> > >> > >> > > > > > >> >> > > > Dmitriy Pavlov
>> > >> > >> > > > > > >> >> > > >
>> > >> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy
>> Pavlov
>> > >> > >> > > > > > >> < [hidden email] >:
>> > >> > >> > > > > > >> >> > > >
>> > >> > >> > > > > > >> >> > > >> Hi Ignite Developers,
>> > >> > >> > > > > > >> >> > > >>
>> > >> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a
>> log
>> > >> with
>> > >> > >> all
>> > >> > >> > > > > commits.
>> > >> > >> > > > > > >> >> > > >>
>> > >> > >> > > > > > >> >> > > >> As far as I can see, nobody removed
>> commits
>> > >> from
>> > >> > >> this
>> > >> > >> > > > sheet,
>> > >> > >> > > > > > so
>> > >> > >> > > > > > >> the
>> > >> > >> > > > > > >> >> > > scope
>> > >> > >> > > > > > >> >> > > >> of release will be discussed in another
>> way:
>> > >> only
>> > >> > >> > > > explicitly
>> > >> > >> > > > > > >> >> declared
>> > >> > >> > > > > > >> >> > > >> commits will be cherry-picked.
>> > >> > >> > > > > > >> >> > > >>
>> > >> > >> > > > > > >> >> > > >> Sincerely,
>> > >> > >> > > > > > >> >> > > >> Dmitriy Pavlov
>> > >> > >> > > > > > >> >> > > >>
>> > >> > >> > > > > > >> >> > >
>> > >> > >> > > > > > >> >> > >
>> > >> > >> > > > > > >> >>
>> > >> > >> > > > > > >> >
>> > >> > >> > > > > >
>> > >> > >> > > > > >
>> > >> > >> > > > > > --
>> > >> > >> > > > > > Zhenya Stanilovsky
>> > >> > >> > > > > >
>> > >> > >> > > >
>> > >> > >> > >
>> > >> > >> >
>> > >> > >>
>> > >> > >
>> > >> >
>> > >>
>> > >
>> >
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Ignite 2.7.5 Release scope

Dmitry Pavlov
Hi Igniters,

Unfortunately, Ignite C++ examples cannot build without
https://issues.apache.org/jira/browse/IGNITE-11195

So this issue should be included in the scope.

Sincerely,
Dmitriy Pavlov

пт, 12 апр. 2019 г. в 13:53, Dmitriy Pavlov <[hidden email]>:

> Hi Igniters,
>
> A bit of update: Last change fixing GridUnsafe for Java 12 was merged to
> master and to 2.7.5
>
> Overnight run contains a number of failures considered as blockers by
> Apache Ignite Teamcity Bot (comparing to 2.7):
> - Platform .NET (Inspections) always failed- Critical F.R.: 100,0%
> - Platform C++ (Linux Clang) & Platform C++ (Linux)* failed in most cases
> - Critical F.R.: 60,0%
> - Disk Page Compressions - Module is N/A in 2.7.5
> - [Inspections] Core - always failed
> - Hibernate 5.3 - Module is N/A in 2.7.5
> - Platform C++ (Win x64 | Release) - also failed in 2.7
>
> And 2 more tests failure considered as blockers (now it is in re-run
> state). If re-run shows these tests are passing I proceed with RC
> preparation.
>
> Sincerely,
> Dmitriy Pavlov
>
> вт, 9 апр. 2019 г. в 10:03, Dmitriy Pavlov <[hidden email]>:
>
>> Hi, Nikolay,
>>
>> thanks for offering help.
>>
>> Performance testing of new fix seems to be almost done.
>>
>> After merging of the last ticket (I hope it will be tomorrow) I will
>> continue with RC building. There can be some issues related to scripts.
>>
>> Sincerely,
>> Dmitriy Pavlov
>>
>> вт, 9 апр. 2019 г. в 09:53, Nikolay Izhikov <[hidden email]>:
>>
>>> Hello, Dmitriy.
>>>
>>> Any news about release?
>>> Do you need assistance with it?
>>>
>>> вт, 2 апр. 2019 г. в 20:04, Dmitriy Pavlov <[hidden email]>:
>>>
>>> > Ivan P., it seems the netty approach you've proposed works well. Thank
>>> you.
>>> >
>>> > Igniters, please take a look at following fix:
>>> > https://github.com/apache/ignite/pull/6384
>>> > It allows us to start under Java 12 and under Java 11- (as it).
>>> >
>>> > Sincerely,
>>> > Dmitriy Pavlov
>>> >
>>> > пт, 29 мар. 2019 г. в 22:57, Dmitriy Pavlov <[hidden email]>:
>>> >
>>> > > Denis, the issue here is that we don't know for sure. We see just one
>>> > > blocking issue with accessing NioAccessObject.
>>> > >
>>> > > And there are 3 different scenario related to this issue fix:
>>> > > - fixes won't help, and we should find out other options on how to
>>> create
>>> > > a direct buffer from pointer - needed for durable memory, Java 12
>>> goes
>>> > to a
>>> > > later release.
>>> > > - some fix would help, but other issues come, Java 12 goes to some
>>> later
>>> > > release
>>> > > - some from proposed fixes works, nothing else needs to be done - 1-2
>>> > days
>>> > >
>>> > > If it latest scenario, I would include as much as we can (1-2 days
>>> extra
>>> > > are comparable with minimal voting time).
>>> > >
>>> > > BTW, I've checked scripts it does not work for me, I will ask Andrey
>>> > > G/Peter for advice on Monday.
>>> > >
>>> > > Sincerely,
>>> > > Dmitriy Pavlov
>>> > >
>>> > >
>>> > > пт, 29 мар. 2019 г. в 20:47, Denis Magda <[hidden email]>:
>>> > >
>>> > >> Folks,
>>> > >>
>>> > >> What are the efforts to support Java 12? Let's do 2.7.6 shortly if
>>> the
>>> > >> fixes are time-consuming.
>>> > >>
>>> > >> -
>>> > >> Denis
>>> > >>
>>> > >>
>>> > >> On Fri, Mar 29, 2019 at 10:08 AM Dmitriy Pavlov <[hidden email]
>>> >
>>> > >> wrote:
>>> > >>
>>> > >> > Hi Igniters,
>>> > >> >
>>> > >> >  I would like to announce code freeze for 2.7.5. Only one open
>>> ticket
>>> > is
>>> > >> > there (reopened):
>>> https://issues.apache.org/jira/browse/IGNITE-11600
>>> > >> (if
>>> > >> > we
>>> > >> > can't start using Java 12 we should clearly state it in
>>> scripts/code).
>>> > >> >
>>> > >> > We're entering to Stabilization phase for
>>> > >> >
>>> > https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.7.5
>>> > >> and
>>> > >> > only blockers may be included into scope. See
>>> > >> >
>>> https://cwiki.apache.org/confluence/display/IGNITE/Release+Process
>>> > for
>>> > >> > more
>>> > >> > details.
>>> > >> >
>>> > >> > Sincerely,
>>> > >> > Dmitriy Pavlov
>>> > >> >
>>> > >> > пт, 29 мар. 2019 г. в 13:51, Dmitriy Pavlov <[hidden email]>:
>>> > >> >
>>> > >> > > Hi Denis,
>>> > >> > >
>>> > >> > > I'm not talking about months. In this discussion, Andrey and
>>> Ivan
>>> > >> > proposed
>>> > >> > > a couple of fixes that may help.
>>> > >> > >
>>> > >> > > It will require a day or two to check if it helps. If it not
>>> helpful
>>> > >> then
>>> > >> > > we should modify startup scripts to say clearly that Java 12 is
>>> not
>>> > >> > > supported.
>>> > >> > >
>>> > >> > > Now under Java 12 Ignite suggests to set startup parameters, but
>>> > even
>>> > >> > with
>>> > >> > > correct parameters, it fails and says please set parameters.
>>> Totally
>>> > >> > > unclear for end-user.
>>> > >> > >
>>> > >> > > I've reopened
>>> https://issues.apache.org/jira/browse/IGNITE-11600
>>> > >> > >
>>> > >> > > Sincerely,
>>> > >> > > Dmitriy Pavlov
>>> > >> > >
>>> > >> > > чт, 28 мар. 2019 г. в 18:51, Denis Magda <[hidden email]>:
>>> > >> > >
>>> > >> > >> If the failure handler improvements will lower down a number of
>>> > >> > >> false-positive cluster shutdowns then I'm for the fix
>>> inclusion to
>>> > >> the
>>> > >> > >> release.
>>> > >> > >>
>>> > >> > >> As for Java 12, I would put to the next release that we can
>>> make
>>> > >> shortly
>>> > >> > >> after this one. We don't need to wait for months if there are
>>> some
>>> > >> > urgent
>>> > >> > >> fixes.
>>> > >> > >>
>>> > >> > >> -
>>> > >> > >> Denis
>>> > >> > >>
>>> > >> > >>
>>> > >> > >> On Thu, Mar 28, 2019 at 8:46 AM Dmitriy Pavlov <
>>> [hidden email]
>>> > >
>>> > >> > >> wrote:
>>> > >> > >>
>>> > >> > >> > Denis, I also +1 for python fix. Today is the planned date of
>>> > code
>>> > >> > >> freeze,
>>> > >> > >> > so today or tomorrow I'll announce it.
>>> > >> > >> >
>>> > >> > >> > Only one thing was suggested is the fix for failure
>>> handlers. I
>>> > >> don't
>>> > >> > >> > deeply understand the pros and cons, but I'm sure we should
>>> > >> courteous
>>> > >> > >> > enough for wait experienced folks to come to a consensus.
>>> > >> > >> >
>>> > >> > >> > One more thing I suggest to check after code freeze, it is
>>> Java
>>> > >> 12. It
>>> > >> > >> does
>>> > >> > >> > not work now, but with thanks to Andrey G and Ivan P. we
>>> probably
>>> > >> have
>>> > >> > >> not
>>> > >> > >> > very complex solution for this.
>>> > >> > >> >
>>> > >> > >> > So these 2 fixes are last candidates to come to scope. WDYT?
>>> > >> > >> >
>>> > >> > >> > Sincerely,
>>> > >> > >> > Dmitriy Pavlov
>>> > >> > >> >
>>> > >> > >> > чт, 28 мар. 2019 г. в 18:10, Denis Magda <[hidden email]
>>> >:
>>> > >> > >> >
>>> > >> > >> > > Yes, the Python example's fix makes sense - that's a
>>> usability
>>> > >> issue
>>> > >> > >> and
>>> > >> > >> > > doesn't affect the core components.
>>> > >> > >> > >
>>> > >> > >> > > In general, I would suggest us to do a code freeze and push
>>> > other
>>> > >> > >> changes
>>> > >> > >> > > to another release. We can't include everything.
>>> > >> > >> > >
>>> > >> > >> > > -
>>> > >> > >> > > Denis
>>> > >> > >> > >
>>> > >> > >> > >
>>> > >> > >> > > On Thu, Mar 28, 2019 at 3:01 AM Ilya Kasnacheev <
>>> > >> > >> > [hidden email]
>>> > >> > >> > > >
>>> > >> > >> > > wrote:
>>> > >> > >> > >
>>> > >> > >> > > > Hello!
>>> > >> > >> > > >
>>> > >> > >> > > > How about adding
>>> > >> > https://issues.apache.org/jira/browse/IGNITE-11366
>>> > >> > >> > > which
>>> > >> > >> > > > fixes Python's release directory?
>>> > >> > >> > > >
>>> > >> > >> > > > I have stumbled on this issue personally.
>>> > >> > >> > > >
>>> > >> > >> > > > Regards,
>>> > >> > >> > > > --
>>> > >> > >> > > > Ilya Kasnacheev
>>> > >> > >> > > >
>>> > >> > >> > > >
>>> > >> > >> > > > чт, 28 мар. 2019 г. в 04:19, Roman Shtykh
>>> > >> > <[hidden email]
>>> > >> > >> >:
>>> > >> > >> > > >
>>> > >> > >> > > > > Dmitriy, the probability of having nodes down with
>>> these
>>> > bugs
>>> > >> > >> under
>>> > >> > >> > > heavy
>>> > >> > >> > > > > load conditions is pretty high. Good candidate for
>>> > >> > cherry-picking.
>>> > >> > >> > > > >
>>> > >> > >> > > > > [1]
>>> https://issues.apache.org/jira/browse/IGNITE-9803[2]
>>> > >> > >> > > > > https://issues.apache.org/jira/browse/IGNITE-11127
>>> > >> > >> > > > > -- Roman
>>> > >> > >> > > > >
>>> > >> > >> > > > >
>>> > >> > >> > > > >     On Wednesday, March 27, 2019, 2:11:49 a.m. GMT+9,
>>> > Dmitriy
>>> > >> > >> Pavlov
>>> > >> > >> > <
>>> > >> > >> > > > > [hidden email]> wrote:
>>> > >> > >> > > > >
>>> > >> > >> > > > >  Hi,
>>> > >> > >> > > > >
>>> > >> > >> > > > > I've cherry-picked this commit. It seems it is critical
>>> > >> because
>>> > >> > it
>>> > >> > >> > also
>>> > >> > >> > > > > fixes storage corruption.
>>> > >> > >> > > > >
>>> > >> > >> > > > > Sincerely,
>>> > >> > >> > > > > Dmitriy Pavlov
>>> > >> > >> > > > >
>>> > >> > >> > > > > вт, 26 мар. 2019 г. в 14:14, Zhenya Stanilovsky
>>> > >> > >> > > > <[hidden email]
>>> > >> > >> > > > > >:
>>> > >> > >> > > > >
>>> > >> > >> > > > > > I suppose this ticket [1] : is very useful too.
>>> > >> > >> > > > > >
>>> > >> > >> > > > > >
>>> > >> > >> > > > > > [1]
>>> https://issues.apache.org/jira/browse/IGNITE-10873 [
>>> > >> > >> > > > > > CorruptedTreeException during simultaneous cache put
>>> > >> > operations
>>> > >> > >> ]
>>> > >> > >> > > > > >
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >------- Forwarded message -------
>>> > >> > >> > > > > > >From: "Alexey Goncharuk" <
>>> [hidden email] >
>>> > >> > >> > > > > > >To: dev < [hidden email] >
>>> > >> > >> > > > > > >Cc:
>>> > >> > >> > > > > > >Subject: Re: Ignite 2.7.5 Release scope
>>> > >> > >> > > > > > >Date: Tue, 26 Mar 2019 13:42:59 +0300
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >Hello Ilya,
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >I do not see any issues with the mentioned test. I
>>> see
>>> > the
>>> > >> > >> > following
>>> > >> > >> > > > > > output
>>> > >> > >> > > > > > >in the logs:
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>>> 21:41:44,970][INFO
>>> > >> > >> > ][main][root]
>>> > >> > >> > > > >>>
>>> > >> > >> > > > > > >Stopping test:
>>> > >> > >> > > > > >
>>> > >> > >> > > > > >
>>> > >> > >> > > > >
>>> > >> > >> > > >
>>> > >> > >> > >
>>> > >> > >> >
>>> > >> > >>
>>> > >> >
>>> > >>
>>> >
>>> >TcpDiscoveryCoordinatorFailureTest#testCoordinatorFailedNoAddFinishedMessageStartOneNode
>>> > >> > >> > > > > > >in 37768 ms <<<
>>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>>> 21:41:44,971][INFO
>>> > >> > >> > ][main][root]
>>> > >> > >> > > > >>>
>>> > >> > >> > > > > > >Stopping test class:
>>> TcpDiscoveryCoordinatorFailureTest
>>> > >> <<<
>>> > >> > >> > > > > > >[21:41:44] : [Step 4/5] [2019-03-22
>>> 21:41:44,971][INFO
>>> > >> > >> > ][main][root]
>>> > >> > >> > > > >>>
>>> > >> > >> > > > > > >Starting test class: IgniteClientConnectTest <<<
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >The issue with Windows may be long connection
>>> timeouts,
>>> > in
>>> > >> > this
>>> > >> > >> > case
>>> > >> > >> > > > we
>>> > >> > >> > > > > > >should either split the suite into multiple ones or
>>> > >> decrease
>>> > >> > >> the
>>> > >> > >> > SPI
>>> > >> > >> > > > > > >timeouts.
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >пн, 25 мар. 2019 г. в 11:24, Ilya Kasnacheev <
>>> > >> > >> > > > > [hidden email]
>>> > >> > >> > > > > > >:
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >> Hello!
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> It seems that I can no longer test this case, on
>>> > >> account of
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > >
>>> > >> > >> > > > >
>>> > >> > >> > > >
>>> > >> > >> > >
>>> > >> > >> >
>>> > >> > >>
>>> > >> >
>>> > >>
>>> >
>>> TcpDiscoveryCoordinatorFailureTest#testClusterFailedNewCoordinatorInitialized
>>> > >> > >> > > > > > >> hanging every time under Java 11 on Windows.
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> Alexey, Ivan, can you please take a look?
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > >
>>> > >> > >> > > > >
>>> > >> > >> > > >
>>> > >> > >> > >
>>> > >> > >> >
>>> > >> > >>
>>> > >> >
>>> > >>
>>> >
>>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> Regards,
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> --
>>> > >> > >> > > > > > >> Ilya Kasnacheev
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> пт, 22 мар. 2019 г. в 16:59, Ilya Kasnacheev <
>>> > >> > >> > > > > > [hidden email] >:
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > > >> > Hello!
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> > Basically there is a test that explicitly
>>> highlights
>>> > >> this
>>> > >> > >> > > problem,
>>> > >> > >> > > > > > that
>>> > >> > >> > > > > > >> is
>>> > >> > >> > > > > > >> > running SSL tests on Windows + Java 11. They
>>> will
>>> > >> hang on
>>> > >> > >> > Master
>>> > >> > >> > > > but
>>> > >> > >> > > > > > >> pass
>>> > >> > >> > > > > > >> > with this patch.
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> > I have started that on TC, results will
>>> probably be
>>> > >> > >> available
>>> > >> > >> > > > later
>>> > >> > >> > > > > > >> today:
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >>
>>> > >> > >> > > > > >
>>> > >> > >> > > > >
>>> > >> > >> > > >
>>> > >> > >> > >
>>> > >> > >> >
>>> > >> > >>
>>> > >> >
>>> > >>
>>> >
>>> https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_SpiWindows&tab=buildTypeStatusDiv&branch_IgniteTests24Java8=__all_branches__
>>> > >> > >> > > > > > >> > (mind the Java version).
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> > Regards,
>>> > >> > >> > > > > > >> > --
>>> > >> > >> > > > > > >> > Ilya Kasnacheev
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> > пт, 22 мар. 2019 г. в 14:13, Maxim Muzafarov <
>>> > >> > >> > > [hidden email]
>>> > >> > >> > > > > >:
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > > >> >> Dmitry, Ilya,
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >> Yes, I've looked through those changes [1] as
>>> they
>>> > >> can
>>> > >> > >> affect
>>> > >> > >> > > my
>>> > >> > >> > > > > > local
>>> > >> > >> > > > > > >> >> PR.  Basically, changes look good to me.
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >> I'm not an expert with CommunicationSpi
>>> component,
>>> > so
>>> > >> > can
>>> > >> > >> > miss
>>> > >> > >> > > > some
>>> > >> > >> > > > > > >> >> details and I haven't tested these changes
>>> under
>>> > Java
>>> > >> > 11.
>>> > >> > >> One
>>> > >> > >> > > > more
>>> > >> > >> > > > > > >> >> thing I'd like to say, I would add additional
>>> tests
>>> > >> to
>>> > >> > PR
>>> > >> > >> > that
>>> > >> > >> > > > will
>>> > >> > >> > > > > > >> >> explicitly highlight the problem being solved.
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >> [1]
>>> > >> https://issues.apache.org/jira/browse/IGNITE-11299
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >> On Thu, 21 Mar 2019 at 22:57, Dmitriy Pavlov <
>>> > >> > >> > > > [hidden email]
>>> > >> > >> > > > > >
>>> > >> > >> > > > > > >> wrote:
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > Hi Igniters,
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > fix
>>> > >> > https://issues.apache.org/jira/browse/IGNITE-11299
>>> > >> > >> > > Avoid
>>> > >> > >> > > > > busy
>>> > >> > >> > > > > > >> wait
>>> > >> > >> > > > > > >> >> on
>>> > >> > >> > > > > > >> >> > processWrite during SSL handshake.
>>> > >> > >> > > > > > >> >> > seems to be blocker cause it is related to
>>> Java
>>> > 11
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > I see Maxim M left some comments. Ilya K.,
>>> Maxim
>>> > >> > M.were
>>> > >> > >> > these
>>> > >> > >> > > > > > >> comments
>>> > >> > >> > > > > > >> >> > addressed?
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > The ticket is in Patch Available. Reviewer
>>> > needed.
>>> > >> > >> Changes
>>> > >> > >> > > > > located
>>> > >> > >> > > > > > >> in
>>> > >> > >> > > > > > >> >> > GridNioServer.
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > Sincerely,
>>> > >> > >> > > > > > >> >> > Dmitriy Pavlov
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > P.S. a quite obvious ticket came to sope, as
>>> > well:
>>> > >> > >> > > > > > >> >> >
>>> > >> https://issues.apache.org/jira/browse/IGNITE-11600
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > чт, 21 мар. 2019 г. в 16:55, Petr Ivanov <
>>> > >> > >> > > [hidden email]
>>> > >> > >> > > > > >:
>>> > >> > >> > > > > > >> >> >
>>> > >> > >> > > > > > >> >> > > Huge +1
>>> > >> > >> > > > > > >> >> > >
>>> > >> > >> > > > > > >> >> > > Will try to add new JDK in nearest time to
>>> our
>>> > >> > >> Teamcity.
>>> > >> > >> > > > > > >> >> > >
>>> > >> > >> > > > > > >> >> > >
>>> > >> > >> > > > > > >> >> > > > On 21 Mar 2019, at 16:27, Dmitriy Pavlov
>>> <
>>> > >> > >> > > > > [hidden email]
>>> > >> > >> > > > > > >
>>> > >> > >> > > > > > >> >> wrote:
>>> > >> > >> > > > > > >> >> > > >
>>> > >> > >> > > > > > >> >> > > > Hi Igniters,
>>> > >> > >> > > > > > >> >> > > >
>>> > >> > >> > > > > > >> >> > > > Meanwhile, Java 12 GA is available. I
>>> suggest
>>> > >> at
>>> > >> > >> least
>>> > >> > >> > > test
>>> > >> > >> > > > > our
>>> > >> > >> > > > > > >> new
>>> > >> > >> > > > > > >> >> tests
>>> > >> > >> > > > > > >> >> > > > scripts with a couple of Java builds.
>>> WDYT?
>>> > >> > >> > > > > > >> >> > > >
>>> > >> > >> > > > > > >> >> > > > Sincerely,
>>> > >> > >> > > > > > >> >> > > > Dmitriy Pavlov
>>> > >> > >> > > > > > >> >> > > >
>>> > >> > >> > > > > > >> >> > > > ср, 20 мар. 2019 г. в 19:21, Dmitriy
>>> Pavlov
>>> > >> > >> > > > > > >> < [hidden email] >:
>>> > >> > >> > > > > > >> >> > > >
>>> > >> > >> > > > > > >> >> > > >> Hi Ignite Developers,
>>> > >> > >> > > > > > >> >> > > >>
>>> > >> > >> > > > > > >> >> > > >> In a separate discussion, I've shared a
>>> log
>>> > >> with
>>> > >> > >> all
>>> > >> > >> > > > > commits.
>>> > >> > >> > > > > > >> >> > > >>
>>> > >> > >> > > > > > >> >> > > >> As far as I can see, nobody removed
>>> commits
>>> > >> from
>>> > >> > >> this
>>> > >> > >> > > > sheet,
>>> > >> > >> > > > > > so
>>> > >> > >> > > > > > >> the
>>> > >> > >> > > > > > >> >> > > scope
>>> > >> > >> > > > > > >> >> > > >> of release will be discussed in another
>>> way:
>>> > >> only
>>> > >> > >> > > > explicitly
>>> > >> > >> > > > > > >> >> declared
>>> > >> > >> > > > > > >> >> > > >> commits will be cherry-picked.
>>> > >> > >> > > > > > >> >> > > >>
>>> > >> > >> > > > > > >> >> > > >> Sincerely,
>>> > >> > >> > > > > > >> >> > > >> Dmitriy Pavlov
>>> > >> > >> > > > > > >> >> > > >>
>>> > >> > >> > > > > > >> >> > >
>>> > >> > >> > > > > > >> >> > >
>>> > >> > >> > > > > > >> >>
>>> > >> > >> > > > > > >> >
>>> > >> > >> > > > > >
>>> > >> > >> > > > > >
>>> > >> > >> > > > > > --
>>> > >> > >> > > > > > Zhenya Stanilovsky
>>> > >> > >> > > > > >
>>> > >> > >> > > >
>>> > >> > >> > >
>>> > >> > >> >
>>> > >> > >>
>>> > >> > >
>>> > >> >
>>> > >>
>>> > >
>>> >
>>>
>>
123