Andrey. > "If it doesn’t work, it doesn’t matter how fast it doesn’t work." (c)
Please, clarify, what do you mean by «doesn’t work»? Are there any unresolved bugs? > IGINTE-12576 affects it minimally All I asking for is to confirm this statement with the benchmark results. > User can disable metrics if it will affect performance. Users can’t disable TcpCommunicationListener nor in any release nor in current master so we should change this code carefully https://github.com/apache/ignite/blob/ignite-2.7.6/modules/core/src/main/java/org/apache/ignite/spi/communication/tcp/TcpCommunicationSpi.java#L1178 > 27 янв. 2020 г., в 20:40, Andrey Gura <ag...@apache.org> написал(а): > > Nikolay, > >> But, we must gather yardstick benchmark results for PR(comparing to current >> master) before merge to ensure there is no performance drop. > > "If it doesn’t work, it doesn’t matter how fast it doesn’t work." (c) > > I believe that benchmarks ignite-2.7.6 vs ignite-2.8 will show > noticeable drop in performance for ignite-2.8. But it is cumulative > effect and IGINTE-12576 affects it minimally. > >> Note, that these metrics updated on each communication message. > > Metrics are not free at all. User can disable metrics if it will > affect performance. > > On Mon, Jan 27, 2020 at 8:23 PM Nikolay Izhikov <nizhi...@apache.org> wrote: >> >> Hello, Andrey. >> >> I’m OK to include these changes to 2.8. >> I don’t review PR, but the ticket description makes sense to me. >> >> But, we must gather yardstick benchmark results for PR(comparing to current >> master) before merge to ensure there is no performance drop. >> Note, that these metrics updated on each communication message. >> >>> 27 янв. 2020 г., в 18:19, Andrey Gura <ag...@apache.org> написал(а): >>> >>> Igniters, >>> >>> I want to add one more issue to the Apache Ignite 2.8 release scope [1]. >>> >>> The problem is impossibility of using communication metrics gathered >>> for nodes in the cluster because node ID will changed in case of >>> restart. Obvious solution is using consistent ID instead of node ID. >>> >>> PR is already implemented and ready for review. >>> >>> [1] https://issues.apache.org/jira/browse/IGNITE-12576 >>> >>> On Fri, Jan 24, 2020 at 4:06 PM Maxim Muzafarov <mmu...@apache.org> wrote: >>>> >>>> Folks, >>>> >>>> >>>> I've cherry-picked these issues [1] [2] to the 2.8 release branch. >>>> >>>> >>>> [1] https://issues.apache.org/jira/browse/IGNITE-12540 >>>> Update versions of vulnerable dependencies >>>> >>>> [2] https://issues.apache.org/jira/browse/IGNITE-12486 >>>> Truncation of archived WAL segments doesn't work >>>> >>>> On Thu, 23 Jan 2020 at 11:08, Ivan Bessonov <bessonov...@gmail.com> wrote: >>>>> >>>>> Hi igniters, >>>>> >>>>> there's a potential data corruption fix that I'd like you to include in >>>>> the >>>>> next release: >>>>> https://issues.apache.org/jira/browse/IGNITE-12486https://issues.apache.org/jira/browse/IGNITE-12486 >>>>> <https://issues.apache.org/jira/browse/IGNITE-12486> >>>>> >>>>> Can you please cherry-pick it? Thank you! >>>>> >>>>> ср, 22 янв. 2020 г. в 17:45, Pavel Tupitsyn <ptupit...@apache.org>: >>>>> >>>>>> Good idea about pre-release build of ignite-2.8 branch. >>>>>> However, I would not name it `rc`, since it is not really a release >>>>>> candidate. Make it `pre0` or something like that. >>>>>> >>>>>> For Ignite.NET I've uploaded pre-release NuGet packages built from >>>>>> current >>>>>> ignite-2.8 branch: >>>>>> https://www.nuget.org/packages/Apache.Ignite/2.8.0-alpha20200122 >>>>>> >>>>>> >>>>>> On Wed, Jan 22, 2020 at 3:09 PM Ilya Kasnacheev >>>>>> <ilya.kasnach...@gmail.com >>>>>>> >>>>>> wrote: >>>>>> >>>>>>> Hello! >>>>>>> >>>>>>> I have committed the bumping of essential dependencies' versions: >>>>>>> https://issues.apache.org/jira/browse/IGNITE-12540 >>>>>>> >>>>>>> Would you mind including this change into the scope of 2.8? No point of >>>>>>> shipping known problematic JARs in our deliverable. >>>>>>> >>>>>>> Regards, >>>>>>> -- >>>>>>> Ilya Kasnacheev >>>>>>> >>>>>>> >>>>>>> ср, 22 янв. 2020 г. в 14:00, Maxim Muzafarov <mmu...@apache.org>: >>>>>>> >>>>>>>> Alexey, >>>>>>>> >>>>>>>> Sure, I've just thought about it too a few days ago. >>>>>>>> >>>>>>>> On Wed, 22 Jan 2020 at 12:09, Anton Vinogradov <a...@apache.org> wrote: >>>>>>>>> >>>>>>>>> Good Idea, this will also check that the release process is alive. >>>>>>>>> >>>>>>>>> On Wed, Jan 22, 2020 at 12:04 PM Alexey Goncharuk < >>>>>>>>> alexey.goncha...@gmail.com> wrote: >>>>>>>>> >>>>>>>>>> Folks, Maxim, >>>>>>>>>> >>>>>>>>>> Do you mind if I build the current state of ignite-2.8 branch and >>>>>>>> upload a >>>>>>>>>> maven staging as rc0 (step 4.3.2 of the release process)? I want >>>>>> run >>>>>>>> some >>>>>>>>>> tests for the fixes that are already included to the branch. >>>>>>>>>> >>>>>>>>>> вт, 21 янв. 2020 г. в 14:28, Maxim Muzafarov <mmu...@apache.org>: >>>>>>>>>> >>>>>>>>>>> Folks, >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> I think both of these issues [1] [2] are critical to 2.8 release >>>>>>> and >>>>>>>>>>> we must include them. >>>>>>>>>>> >>>>>>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-12547 >>>>>>>>>>> Excessive AtomicLong instantiations lead to GC pressure. >>>>>>>>>>> >>>>>>>>>>> [2] https://issues.apache.org/jira/browse/IGNITE-12530 >>>>>>>>>>> Pages list caching can cause IgniteOOME when the checkpoint is >>>>>>>>>>> triggered by "too many dirty pages" reason. >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> On Mon, 20 Jan 2020 at 19:00, Alex Plehanov < >>>>>>> plehanov.a...@gmail.com >>>>>>>>> >>>>>>>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>> Guys, >>>>>>>>>>>> >>>>>>>>>>>> There is an issue [1] caused by page list caching [2], which >>>>>> also >>>>>>>>>> affects >>>>>>>>>>>> 2.8 release. IgniteOutOfMemoryException can be thrown in some >>>>>>> cases >>>>>>>>>> (data >>>>>>>>>>>> region is small, a checkpoint is triggered by "too many dirty >>>>>>>> pages" >>>>>>>>>>> reason >>>>>>>>>>>> and pages list cache is rather big). >>>>>>>>>>>> The fix is ready and merged to master, I suggest to include >>>>>> this >>>>>>>> fix to >>>>>>>>>>> 2.8 >>>>>>>>>>>> release. What do you think? >>>>>>>>>>>> >>>>>>>>>>>> [1]: https://issues.apache.org/jira/browse/IGNITE-12530 >>>>>>>>>>>> [2]: https://issues.apache.org/jira/browse/IGNITE-6930 >>>>>>>>>>>> >>>>>>>>>>>> пн, 20 янв. 2020 г. в 12:57, Alexey Goncharuk < >>>>>>>>>>> alexey.goncha...@gmail.com>: >>>>>>>>>>>> >>>>>>>>>>>>> Maxim, >>>>>>>>>>>>> >>>>>>>>>>>>> I took a quick look at IGNITE-12456 and I am not sure it's >>>>>>> about >>>>>>>> data >>>>>>>>>>>>> corruption. In the attached logs blocked system threads are >>>>>>>> reported, >>>>>>>>>>>>> however, there is no enough information to investigate the >>>>>>> issue >>>>>>>> (the >>>>>>>>>>> full >>>>>>>>>>>>> thread dump was not attached). I asked the ticket creator to >>>>>>>> attach >>>>>>>>>>> missing >>>>>>>>>>>>> pieces. >>>>>>>>>>>>> >>>>>>>>>>>>> Should we consider moving this ticket to a next release? >>>>>>>>>>>>> >>>>>>>>>>>>> пн, 20 янв. 2020 г. в 08:54, Zhenya Stanilovsky >>>>>>>>>>> <arzamas...@mail.ru.invalid >>>>>>>>>>>>>> : >>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> Maxim, performance fix issue [1] already in master, if no >>>>>>>>>>> objections, can >>>>>>>>>>>>>> u merge it into 2.8 ? Thanks ! >>>>>>>>>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-12547 >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Igniters, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Here is the actual list of BLOCKER release issues: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> IGNITE-12456 Cluster Data Store grid gets Corrupted for >>>>>> Load >>>>>>>> test >>>>>>>>>>>>>>> *[Unassigned]* OPEN >>>>>>>>>>>>>>> IGNITE-12489 Error during purges by expiration: Unknown >>>>>> page >>>>>>>> type* >>>>>>>>>>>>>>> [Unassigned]* OPEN >>>>>>>>>>>>>>> IGNITE-8641 SpringDataExample should use >>>>>> example-ignite.xml >>>>>>>> config >>>>>>>>>>>>>>> *[Unassigned]* OPEN >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> IGNITE-12398 Apache Ignite Cluster(Amazon S3 Based >>>>>>> Discovery) >>>>>>>>>> Nodes >>>>>>>>>>>>>> getting >>>>>>>>>>>>>>> down [Emmanouil Gkatziouras] OPEN >>>>>>>>>>>>>>> IGNITE-9184 Cluster hangs during concurrent node client >>>>>> and >>>>>>>> server >>>>>>>>>>> nodes >>>>>>>>>>>>>>> restart [Dmitriy Sorokin] IN PROGRESS >>>>>>>>>>>>>>> IGNITE-12553 [IEP-35] public Java metric API Improvement >>>>>>>> [Nikolay >>>>>>>>>>>>> Izhikov] >>>>>>>>>>>>>>> Blocker IN PROGRESS >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> IGNITE-12227 Default auto-adjust baseline enabled flag >>>>>>>> calculated >>>>>>>>>>>>>>> incorrectly [Anton Kalashnikov] PATCH AVAILABLE >>>>>>>>>>>>>>> IGNITE-12470 Pme-free switch feature should be >>>>>> deactivatable >>>>>>>>>> [Sergei >>>>>>>>>>>>>>> Ryzhov] PATCH AVAILABLE >>>>>>>>>>>>>>> IGNITE-12552 [IEP-35] Expose MetricRegistry to the public >>>>>>> API >>>>>>>>>>>>> Improvement >>>>>>>>>>>>>>> [Nikolay Izhikov] PATCH AVAILABLE >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-12456 >>>>>>>>>>>>>>> [2] https://issues.apache.org/jira/browse/IGNITE-12489 >>>>>>>>>>>>>>> [3] https://issues.apache.org/jira/browse/IGNITE-8641 >>>>>>>>>>>>>>> [8] https://issues.apache.org/jira/browse/IGNITE-12398 >>>>>>>>>>>>>>> [3] https://issues.apache.org/jira/browse/IGNITE-9184 >>>>>>>>>>>>>>> [6] https://issues.apache.org/jira/browse/IGNITE-12553 >>>>>>>>>>>>>>> [7] https://issues.apache.org/jira/browse/IGNITE-12227 >>>>>>>>>>>>>>> [9] https://issues.apache.org/jira/browse/IGNITE-12470 >>>>>>>>>>>>>>> [5] https://issues.apache.org/jira/browse/IGNITE-12552 >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Sat, 18 Jan 2020 at 19:11, Sergey Antonov < >>>>>>>>>>>>> antonovserge...@gmail.com >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Conflicts in pr [1] are resolved. TC Run all is started. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> [1] https://github.com/apache/ignite/pull/7238 >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> пт, 17 янв. 2020 г. в 16:04, Sergey Antonov < >>>>>>>>>>>>> antonovserge...@gmail.com >>>>>>>>>>>>>>> : >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> I will do that on monday (20/01). >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> пт, 17 янв. 2020 г. в 13:08, Maxim Muzafarov < >>>>>>>>>> mmu...@apache.org >>>>>>>>>>>> : >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Sergey, >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Can you, please, resolve the PR conflicts [1] [2]? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> [1] https://github.com/apache/ignite/pull/7238 >>>>>>>>>>>>>>>>>> [2] >>>>>> https://issues.apache.org/jira/browse/IGNITE-11256 >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> On Thu, 16 Jan 2020 at 16:59, Ilya Kasnacheev < >>>>>>>>>>>>>> ilya.kasnach...@gmail.com > >>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Hello! >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I have bumped beanutils and re-ran Cassandra Store >>>>>>>> tests. >>>>>>>>>> Can >>>>>>>>>>> you >>>>>>>>>>>>>>>>>> please >>>>>>>>>>>>>>>>>>> comment on the ticket? >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I think that fixing ZooKeeper is too much effort >>>>>>>> (there's >>>>>>>>>>> chaos >>>>>>>>>>>>> with >>>>>>>>>>>>>>>>>>> jackson vs. jackson-asl), maybe it should be split >>>>>> up >>>>>>>> as a >>>>>>>>>>>>> separate >>>>>>>>>>>>>>>>>> ticket >>>>>>>>>>>>>>>>>>> to be done later. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Regards, >>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>> Ilya Kasnacheev >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> ср, 15 янв. 2020 г. в 18:31, Vladimir Pligin < >>>>>>>>>>>>> vova199...@yandex.ru >>>>>>>>>>>>>>> : >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Thanks, Ilya. It would be really great to have >>>>>> your >>>>>>>> patch >>>>>>>>>>>>> included >>>>>>>>>>>>>>>>>> into 2.8 >>>>>>>>>>>>>>>>>>>> scope. >>>>>>>>>>>>>>>>>>>> I'd like to give my two cent as well. For example >>>>>> we >>>>>>>> have >>>>>>>>>>>>>> vulnerable >>>>>>>>>>>>>>>>>>>> dependencies here: >>>>>>>>>>>>>>>>>>>> modules/cassandra/store/pom.xml - >>>>>> commons-beanutils >>>>>>>>>>>>>>>>>>>> modules/zookeeper/pom.xml - transitive Jackson >>>>>> from >>>>>>>>>> Curator >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> I'd suggest to uprgrade >>>>>>>>>> commons-beanutils:commons-beanutils >>>>>>>>>>> to >>>>>>>>>>>>>> 1.9.4 >>>>>>>>>>>>>>>>>> and >>>>>>>>>>>>>>>>>>>> override >>>>>> com.fasterxml.jackson.core:jackson-databind >>>>>>>> to >>>>>>>>>> our >>>>>>>>>>>>> common >>>>>>>>>>>>>>>>>> jackson >>>>>>>>>>>>>>>>>>>> version from other modules. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>> Sent from: >>>>>>>>>>>>>> http://apache-ignite-developers.2346864.n4.nabble.com/ >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>> BR, Sergey Antonov >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>> BR, Sergey Antonov >>>>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> Sincerely yours, >>>>> Ivan Bessonov >>