+1 for the plan

> 2 дек. 2020 г., в 16:21, Maxim Muzafarov <mmu...@apache.org> написал(а):
> 
> 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 <mr.wei...@gmail.com> 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 <nizhi...@apache.org> 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, Вячеслав Коптилин <slava.kopti...@gmail.com> 
>>>> написал(а):
>>>> 
>>>> 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 <nizhi...@apache.org>:
>>>> 
>>>>> 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, Вячеслав Коптилин <slava.kopti...@gmail.com>
>>>>> написал(а):
>>>>>> 
>>>>>> 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 <nizhi...@apache.org>:
>>>>>> 
>>>>>>> +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 <mmu...@apache.org>
>>>>> написал(а):
>>>>>>>> 
>>>>>>>> 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, Вячеслав Коптилин <
>>>>> slava.kopti...@gmail.com>
>>>>>>> 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.
>>>>>>> 
>>>>>>> 
>>>>> 
>>>>> 
>>> 
>> 

Reply via email to