Pert, I think we should start with the vote. So, the plan can be:
1. Vote for MVCC removal (if success then next steps) 2. Disable MVCC with some patch. 3. Turn off tests on TC. 4. Create issues with full code removal. On Wed, 2 Dec 2020 at 16:03, Petr Ivanov <[email protected]> wrote: > > How about: > > > 1. Disable MVCC with some patch. > 2. Turn off tests. > 3. Plan full code removal. > > ? > > > On 2 Dec 2020, at 16:01, Nikolay Izhikov <[email protected]> wrote: > > > >> 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. > >>>>> > >>>>> > >>> > >>> > > >
