Hi Anton, +1 for removal MVCC
I think it would be nice to close (with "will not fix" status) all open tickets related to MVCC: https://issues.apache.org/jira/browse/IGNITE-12118?jql=project%20%3D%20IGNITE%20AND%20resolution%20%3D%20Unresolved%20%20and%20summary%20~%20%22MVCC%22%20ORDER%20BY%20%20priority%20DESC%2C%20updated%20DESC WDYT? Thanks, Slava. вт, 22 авг. 2023 г. в 22:03, Anton Vinogradov <a...@apache.org>: > Going to merge on green visa obtaining if there are no objections. > > On Tue, Aug 22, 2023 at 6:47 PM Anton Vinogradov <a...@apache.org> wrote: > > > MVCC test/suites removal PR [1] is almost ready, fixing final failures. > > > > Does anybody ready to review the PR? > > > > I've rechecked each modified line before pushing the commit button, is > > this enough? > > > > [1] https://github.com/apache/ignite/pull/10900 > > > > On Thu, Aug 17, 2023 at 1:18 PM Anton Vinogradov <a...@apache.org> wrote: > > > >> Great, starting the removal. > >> > >> On Wed, Aug 16, 2023 at 5:35 PM Ivan Daschinsky <ivanda...@gmail.com> > >> wrote: > >> > >>> The plan looks good to me. Some of the tests are in the ODBC test > suite, > >>> so > >>> i can help if needed. > >>> > >>> ср, 16 авг. 2023 г. в 16:32, Anton Vinogradov <a...@apache.org>: > >>> > >>> > Igniters, > >>> > > >>> > I started the TX code cleanup [1] last month and almost finished with > >>> the > >>> > obvious garbage. > >>> > Now, started the code deduplication, I was faced with code > >>> overcomplexity > >>> > because of unfinished MVCC. > >>> > > >>> > The community agreed to remove MVCC, but the initial attempt [2] was > >>> not > >>> > successful because of the impossibility to get rid of 20k+ lines of > the > >>> > code at once. > >>> > So, my proposal is to remove it step by step. > >>> > > >>> > 1) MVCC tests should be removed from the project > >>> > 2) MVCC-related code should be removed from the project by reasonably > >>> sized > >>> > commits, checking it does not affect the existing tests. > >>> > > >>> > I'm ready to perform the removal. > >>> > > >>> > Any objections/tips? > >>> > > >>> > [1] https://issues.apache.org/jira/browse/IGNITE-19844 > >>> > [2] https://issues.apache.org/jira/browse/IGNITE-13871 > >>> > > >>> > >>> > >>> -- > >>> Sincerely yours, Ivan Daschinskiy > >>> > >> >
