ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Maksim Timonin <timonin.ma...@gmail.com>
Subject [DISCUSSION] Patch completely breaks MVCC.
Date Mon, 15 Mar 2021 18:58:04 GMT
Hi, Igniters!

I'm working on a feature (moving indexes to the core module) and skip
specific implementation for MVCC as it is considered deprecated (the vote
result [1]). Am I right that now there is no need to support MVCC? Then
there are a lot of tests (both Java, C++) that fail because they run with
TRANSACTIONAL_SNAPSHOT atomicity mode.

There are 2 cases:
1. MVCC mode is just a parameter of a test. I just removed it from a
parameters list;
2. There are tests that run only for MVCC. I marked them with the @Ignore
annotation.

But would it better just completely remove all such tests that are broken
by the patch?

[1]
http://apache-ignite-developers.2346864.n4.nabble.com/RESULT-VOTE-Removing-MVCC-public-API-td50705.html#a50706

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message