Slava, Can you start the vote?
It's strange turning off here the whole MVCC tests just because something in the master branch was broken when in the second thread Community decide to continue MVCC support. Let's start the vote and see what happens. On Wed, 2 Dec 2020 at 16:39, Вячеслав Коптилин <[email protected]> wrote: > > > It will be even worse if our users will face NPE or things like that in > the basic MVCC scenarios just because we don’t tests it. > The feature is not production-ready, and I don't think it is used at all. > Moreover, MVCC Cache 7, 8, 8, MVCC PDS 1, 2, 4 are already broken > (execution timeouts, flaky test, etc) and I haven't seen anyone who would > like to fix this. > Why should we waste every contributor's time? IMHO, MVCC suites are useless > and everyone just pushes "re-run possible blockers" button and doesn't care > about MVCC tests at all. > > Thanks, > S. > > ср, 2 дек. 2020 г. в 16:01, Nikolay Izhikov <[email protected]>: > > > > I think test suites can be disabled even today > > > > I’m -1 to disable tests without complete removal. > > It will be even worse if our users will face NPE or things like that in > > the basic MVCC scenarios just because we don’t tests it. > > > > > > > 2 дек. 2020 г., в 15:50, Вячеслав Коптилин <[email protected]> > > написал(а): > > > > > > Hi Nikolay, > > > > > >> Why do we need feature in the project that not even tested regularly? > > > Fair enough. However, I am not an expert in this area (MVCC and SQL), so > > I > > > cannot say how much effort it will take. > > > I would say that the opinion of the rest of the community is needed here. > > > > > > Anyway, I think test suites can be disabled even today, while the fate of > > > the MVCC feature can be (and should be) discussed separately. > > > What do you think? > > > > > > Thanks, > > > S. > > > > > > ср, 2 дек. 2020 г. в 15:38, Nikolay Izhikov <[email protected]>: > > > > > >> Hello, Slava! > > >> > > >> Yes, this topic comes to the top from time to time :) > > >> > > >>> . I just want to save the time required for getting TCBot's visa and TC > > >> resources. > > >> > > >> Why do we need feature in the project that not even tested regularly? > > >> > > >>> 2 дек. 2020 г., в 15:36, Вячеслав Коптилин <[email protected]> > > >> написал(а): > > >>> > > >>> Hello Nikolay, > > >>> > > >>>> +1 to vote for complete MVCC removal. > > >>> It has already been discussed here [1] and, unfortunately, I have not > > >> seen > > >>> an agreement on that. > > >>> > > >>> [1] > > >>> > > >> > > http://apache-ignite-developers.2346864.n4.nabble.com/Mark-MVCC-with-IgniteExperimental-td45669.html > > >>> > > >>> > > >>> Thanks, > > >>> S. > > >>> > > >>> ср, 2 дек. 2020 г. в 13:05, Nikolay Izhikov <[email protected]>: > > >>> > > >>>> +1 to vote for complete MVCC removal. > > >>>> > > >>>> MVCC is a great feature but we should implement it as a first-class > > >>>> feature and not «something that pretends to be working» > > >>>> > > >>>>> 2 дек. 2020 г., в 12:53, Maxim Muzafarov <[email protected]> > > >> написал(а): > > >>>>> > > >>>>> Hello Slava, > > >>>>> > > >>>>> I think we should vote for MVCC termination of support. If the vote > > >>>>> will be successful than remove it from the source code and disable > > >>>>> MVCC suites. > > >>>>> > > >>>>> Only disabling tests from MVCC sounds not good. > > >>>>> > > >>>>> On Wed, 2 Dec 2020 at 12:32, Вячеслав Коптилин < > > >> [email protected]> > > >>>> wrote: > > >>>>>> > > >>>>>> Hello Igniters, > > >>>>>> > > >>>>>> It looks like there is no activity related to maintaining or > > >> developing > > >>>> the > > >>>>>> MVCC feature. > > >>>>>> So, I see no reason to waste TeamCity resources. I propose to > > disable > > >>>> the > > >>>>>> corresponding test suites. > > >>>>>> This has already been discussed here as well [1]. > > >>>>>> > > >>>>>> [1] > > >>>>>> > > >>>> > > >> > > http://apache-ignite-developers.2346864.n4.nabble.com/Mark-MVCC-with-IgniteExperimental-td45669.html > > >>>>>> > > >>>>>> Thanks, > > >>>>>> S. > > >>>> > > >>>> > > >> > > >> > > > >
