Guys, are you targeting this for the next big Ignite release? Should be in 3 m from now.
-- Denis On Thu, Jun 14, 2018 at 2:58 AM Anton Vinogradov <a...@apache.org> wrote: > Corrected IEP URL: > > https://cwiki.apache.org/confluence/display/IGNITE/IEP-21%3A+JCache+1.1+support > > чт, 14 июн. 2018 г. в 12:48, Александр Меньшиков <sharple...@gmail.com>: > > > Igniters, > > > > I've prepared IEP-21 [1] for this JCache updating task. > > It will help us to manage the issues and see the progress in one place. > > Also, we have finally added tests for TCK 1.1.0 [2] to our TC which can > be > > run on any branch. > > Both tests cases (for 1.0.1 and for 1.1.0) will coexist until IEP-21 > > finish. > > > > [1] > https://cwiki.apache.org/confluence/display/IGNITE/IEP-21:+JCache+1.1 > > [2] > > > > > https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_JCacheTck11 > > > > 2018-06-06 0:49 GMT+03:00 Denis Magda <dma...@gridgain.com>: > > > > > Agree, I see zero benefits of being compliant with both specification > > > versions. Let’s just focus on the latest one. > > > > > > Denis > > > > > > On Tuesday, June 5, 2018, Dmitriy Setrakyan <dsetrak...@apache.org> > > wrote: > > > > > > > Alex, > > > > > > > > I think it is OK to break TCK 1.0.1 tests in favor of TCK 1.1. Once > we > > > > finish the migration, I would remove the TCK 1.0.1 test suite > > altogether. > > > > > > > > D. > > > > > > > > On Tue, Jun 5, 2018 at 11:13 AM, Александр Меньшиков < > > > sharple...@gmail.com > > > > > > > > > wrote: > > > > > > > > > Okay. There are tests results: > > > > > > > > > > https://ci.ignite.apache.org/viewLog.html?buildId=1361493& > > > > > tab=buildResultsDiv&buildTypeId=IgniteTests24Java8_JCacheTck11 > > > > > > > > > > It's the same as locally. > > > > > > > > > > Also, I have created sub-tasks for all problems we have: > > > > > > > > > > 1) CacheManagerTest.getUnsafeTypedCacheRequest failed. > > > > > https://issues.apache.org/jira/browse/IGNITE-8704 > > > > > > > > > > 2) CacheMBStatisticsBeanTest.testClear failed. > > > > > https://issues.apache.org/jira/browse/IGNITE-8705 > > > > > > > > > > 3) CacheManagerTest.close_cachesEmpty failed. > > > > > https://issues.apache.org/jira/browse/IGNITE-8708 > > > > > > > > > > 4) CacheMBStatisticsBeanTest.testPutIfAbsent failed. > > > > > https://issues.apache.org/jira/browse/IGNITE-8709 > > > > > > > > > > 5) CacheEntryEvent.getOldValue should be available. > > > > > Two tests fail because of it. > > > > > > > > > > Looks like a bug. > > > > > > > > > > https://issues.apache.org/jira/browse/IGNITE-8714 > > > > > > > > > > 6) Problems with Closeable objects from Factory > > > > > > > > > > *98* tests fail because of it. > > > > > > > > > > https://issues.apache.org/jira/browse/IGNITE-8715 > > > > > > > > > > > > > > > For first 4 problems, I already have PRs. > > > > > Problems 2) and 3) will break tests for TCK 1.0.1 because these > tests > > > > work > > > > > wrong in 1.0.1, > > > > > and were fixed in 1.1.0. > > > > > > > > > > 2018-06-05 14:37 GMT+03:00 Dmitry Pavlov <dpavlov....@gmail.com>: > > > > > > > > > > > Agree with Nikolay we should create build plan, and we can use > this > > > > build > > > > > > plan in developement branch. > > > > > > > > > > > > Merge to master is not necessary before issue is ready. > > > > > > > > > > > > вт, 5 июн. 2018 г. в 14:04, Nikolay Izhikov <nizhi...@apache.org > >: > > > > > > > > > > > >> Alex, please try to run this build plan for your branch > > > > > >> > > > > > >> https://ci.ignite.apache.org/viewType.html?buildTypeId= > > > > > >> IgniteTests24Java8_JCacheTck11 > > > > > >> > > > > > >> В Вт, 05/06/2018 в 13:56 +0300, Nikolay Izhikov пишет: > > > > > >> > Guys, we had a private talk with Vyacheslav and Dmitriy > Pavlov. > > > > > >> > Here are its resulst: > > > > > >> > > > > > > >> > 1. I will create JCache1.1 build plan. > > > > > >> > I will be able to run tests with Alex new profile enabled. > > > > > >> > > > > > > >> > So, Alex can run and share with community tests results both > for > > > > > jcache > > > > > >> 1.0 and jcache 1.1. > > > > > >> > > > > > > >> > 2. Alex, please, create tickets for a JCache 1.1. issues. > > > > > >> > > > > > > >> > > > > > > >> > В Вт, 05/06/2018 в 13:36 +0300, Vyacheslav Daradur пишет: > > > > > >> > > AFAIK TeamCity is not able to create such build-plan on the > > fly. > > > > > >> > > Moreover, we will not be able to test master branch in both > > > > profiles > > > > > >> > > in this case. > > > > > >> > > > > > > > >> > > Am I miss something? > > > > > >> > > > > > > > >> > > On Tue, Jun 5, 2018 at 1:31 PM, Nikolay Izhikov < > > > > > nizhi...@apache.org> > > > > > >> wrote: > > > > > >> > > > Vyacheslav, > > > > > >> > > > > > > > > >> > > > Let's create build plan on TC for this profile. > > > > > >> > > > Why we need to merge it in master now? > > > > > >> > > > > > > > > >> > > > В Вт, 05/06/2018 в 13:29 +0300, Vyacheslav Daradur пишет: > > > > > >> > > > > Nikolay, there isn't anything broken in PR. > > > > > >> > > > > > > > > > >> > > > > The PR is needed to add new build-plan on TC. > > > > > >> > > > > > > > > > >> > > > > We need tools to check that fixes for 1.1 don't break > > > > > >> compatibility with 1.0. > > > > > >> > > > > > > > > > >> > > > > On Tue, Jun 5, 2018 at 1:21 PM, Кузнецов Алексей Львович > > > > > >> > > > > <alkuznetsov...@gmail.com> wrote: > > > > > >> > > > > > Hi > > > > > >> > > > > > > > > > > >> > > > > > After Alexander create separate tickets for failed > > tests, > > > > > >> everybody is free > > > > > >> > > > > > to fix them. > > > > > >> > > > > > So we can proceed faster issue resolving. > > > > > >> > > > > > > > > > > >> > > > > > > Hello, Igniters. > > > > > >> > > > > > > > > > > > >> > > > > > > Actually, I don't understand why we should merge in > > > master > > > > > >> something > > > > > >> > > > > > > broken. > > > > > >> > > > > > > Currently, Ignite is not ready for JCache 1.1. > > > > > >> > > > > > > > > > > > >> > > > > > > Only change I see in PR is new profile [1]. > > > > > >> > > > > > > Is it required to have it to continue jcache 1.1 > > support > > > > > >> implementation? > > > > > >> > > > > > > > > > > > >> > > > > > > I think Alexandex can proceed with current profile > and > > > > > change > > > > > >> it to run > > > > > >> > > > > > > tests for JCache 1.1 his own branch. > > > > > >> > > > > > > > > > > > >> > > > > > > Am I miss something? > > > > > >> > > > > > > > > > > > >> > > > > > > [1] https://github.com/apache/ignite/pull/4114 > > > > > >> > > > > > > > > > > > >> > > > > > > В Вт, 05/06/2018 в 12:50 +0300, Dmitry Pavlov пишет: > > > > > >> > > > > > > > > > > > > >> > > > > > > > Hi Alexander, > > > > > >> > > > > > > > > > > > > >> > > > > > > > https://issues.apache.org/jira/browse/IGNITE-8687 > > is > > > > 'In > > > > > >> progress'. Is it > > > > > >> > > > > > > > expected? > > > > > >> > > > > > > > > > > > > >> > > > > > > > Nikolay, have you some time to apply patch, if it > > > passes > > > > > >> review? > > > > > >> > > > > > > > > > > > > >> > > > > > > > Sincerely, > > > > > >> > > > > > > > Dmitriy Pavlov > > > > > >> > > > > > > > > > > > > >> > > > > > > > вт, 5 июн. 2018 г. в 5:09, Dmitriy Setrakyan < > > > > > >> dsetrak...@apache.org>: > > > > > >> > > > > > > > > > > > > >> > > > > > > > > Thanks, Alex! Sounds like a good plan. > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > On Mon, Jun 4, 2018 at 5:52 AM, Александр > > Меньшиков > > > > > >> > > > > > > > > <sharple...@gmail.com> > > > > > >> > > > > > > > > wrote: > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > > Hi, > > > > > >> > > > > > > > > > Igniters! > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > I have taken a look at the jcache 1.1 spec and > > > TCK. > > > > > >> > > > > > > > > > And I can write a brief summary of my plan to > > > solve > > > > > the > > > > > >> task. > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > I have found 6 problems in current master with > > TCK > > > > 1.1 > > > > > >> (104 failed > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > tests). > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > Of course, we should run this TCK on CI to be > > > > > >> absolutely sure we didn't > > > > > >> > > > > > > > > > miss something. > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > So the first step is an adding TCK 1.1 suite > to > > > our > > > > > >> Team City. > > > > > >> > > > > > > > > > I have created sub-task [1] for it and > prepared > > > the > > > > PR > > > > > >> [2]. > > > > > >> > > > > > > > > > I need someone with access to merge PR and add > > > suite > > > > > to > > > > > >> Team City. > > > > > >> > > > > > > > > > It going to be just a clone of the current > > > > jcache-tck > > > > > >> suite, but with > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > using > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > the new profile. > > > > > >> > > > > > > > > > You can test new profile locally with the > > > following > > > > > >> command: > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > mvn test -P-release,jcache-tck-1.1 -pl > > > :ignite-core > > > > > -am > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > After that, I will start to add sub-task for > > every > > > > > >> problem we have. > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > Nikolay, can you please help me with merging > [1] > > > and > > > > > >> adding to the > > > > > >> > > > > > > > > > suite? > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > [1]JIRA: https://issues.apache.org/ > > > > > >> jira/browse/IGNITE-8687 > > > > > >> > > > > > > > > > [2]PR: https://github.com/apache/ > > > > > ignite/pull/4114/files > > > > > >> > > > > > > > > > [3]CI: > > > > > >> > > > > > > > > > https://ci.ignite.apache.org/ > > > > > viewType.html?buildTypeId= > > > > > >> > > > > > > > > > IgniteTests24Java8_RunAll& > > > > branch_IgniteTests24Java8= > > > > > >> pull/4114/head&tab= > > > > > >> > > > > > > > > > buildTypeStatusDiv > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > 2018-05-23 14:31 GMT+03:00 Александр > Меньшиков < > > > > > >> sharple...@gmail.com>: > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > > Thanks, Slava. You are right. > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > > > > 2018-05-23 14:00 GMT+03:00 Vyacheslav > Daradur > > < > > > > > >> daradu...@gmail.com>: > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > > > > > Hi, Alex! > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > Please have a look at maven profile named > > > > > >> "jcache-tck". > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > I believe this is what you are looking > for. > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > On Wed, May 23, 2018 at 11:50 AM, > Александр > > > > > >> Меньшиков > > > > > >> > > > > > > > > > > > <sharple...@gmail.com> wrote: > > > > > >> > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > Igniters, > > > > > >> > > > > > > > > > > > > I think I can do it. As I see we already > > > have > > > > > >> JCache TCK tests in > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > TC. > > > > > >> > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > Can I take somewhere settings/script > which > > > we > > > > > are > > > > > >> using? > > > > > >> > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > 2018-05-23 2:58 GMT+03:00 Dmitriy > > Setrakyan > > > < > > > > > >> dsetrak...@apache.org > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > : > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > Igniters, > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > It will be great if someone in the > > > community > > > > > >> would pick this up. > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > The > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > amount > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > of changes are minimal and many of > them > > > only > > > > > >> have to do with > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > clarifying the > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > documentation. However, removing JSR > 107 > > > > > >> license confusion in 1.1 > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > would be > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > great for Ignite. > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > D. > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > On Tue, May 22, 2018 at 3:04 PM, Denis > > > > Magda < > > > > > >> dma...@apache.org> > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > wrote: > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > Here is a list of all changes: > > > > > >> > > > > > > > > > > > > > > https://groups.google.com/ > > > > > >> forum/#!topic/jsr107/BC1qKqknzKU > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > The primary argument for the > migration > > > is > > > > a > > > > > >> license. JCache 1.0 > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > is > > > > > >> > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > licensed > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > by Oracle that causes legal issues > for > > > > some > > > > > >> of the users. Once we > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > upgrade > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > to JCache 1.1 the won't longer be a > > big > > > > > deal. > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > However, once we move to 1.1 we need > > to > > > be > > > > > >> sure that we comply > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > with > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > the > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > updated specification. > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > -- > > > > > >> > > > > > > > > > > > > > > Denis > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > On Tue, May 22, 2018 at 5:20 AM, > > Dmitry > > > > > >> Pavlov < > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > dpavlov....@gmail.com> > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > wrote: > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > Hi Denis, > > > > > >> > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > What was improved in JCache 1.1? > > > > > >> > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > Would it be useful for product to > > > change > > > > > >> supported spec. > > > > > >> > > > > > > > > > > > > > >> > > > > > > > > version? > > > > > >> > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > Sincerely, > > > > > >> > > > > > > > > > > > > > > > Dmitriy Pavlov > > > > > >> > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > пн, 21 мая 2018 г. в 20:12, Denis > > > Magda > > > > < > > > > > >> dma...@apache.org>: > > > > > >> > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > Igniters, > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > Eventually, JCache was > relicensed > > to > > > > > >> Apache 2.0 and released > > > > > >> > > > > > > > > > > > > > > >> > > > > > > > > > 1.1 > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > version: > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > https://groups.google.com/ > > > > > >> forum/#!topic/jsr107/BC1qKqknzKU > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > Is there anyone interested in > > > > upgrading > > > > > >> Ignite to the new > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > version for > > > > > >> > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > the > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > next release? > > > > > >> > > > > > > > > > > > > > > > > https://issues.apache.org/ > > > > > >> jira/browse/IGNITE-8548 > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > -- > > > > > >> > > > > > > > > > > > > > > > > Denis > > > > > >> > > > > > > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > > > -- > > > > > >> > > > > > > > > > > > Best Regards, Vyacheslav D. > > > > > >> > > > > > > > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > >