Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
MIME-Version: 1.0
In-Reply-To: <6B66EC0C-E331-4815-9BDD-C3C333CA0FAE@gridgain.com>
References: <CAGcMBHjex5heWt5UJrkcuE3STswX6SZ0LiJbxvb0fj+KTWgxFg@mail.gmail.com>
 <CAMbfdjY-nSM4XuRq0-H8PxiRoKKKgwoLTmB87Ya8CkqPd3KaAg@mail.gmail.com>
 <CABDss3hK3vy56cN8prf9evk7J74uJKjCB=_RnXxtDO4-EFnA7A@mail.gmail.com>
 <CAGcMBHjgxb_=p0piAzqFcUJqTZrmtXwhjyHUDLLySidzqJTWQQ@mail.gmail.com>
 <CA+0=VoUK+0g6RsKyegZcGoLg5EaUEiPC83KXyphScxs-CU=AwQ@mail.gmail.com> <6B66EC0C-E331-4815-9BDD-C3C333CA0FAE@gridgain.com>
From: Dmitriy Setrakyan <dsetrakyan@apache.org>
Date: Mon, 7 Nov 2016 13:46:17 -0800
Message-ID: <CA+0=VoXO0n2iwVJcnKsKFUEUyLgAbUaFOaCjwZzrACMduAUgzQ@mail.gmail.com>
Subject: Re: TX savepoints
To: dev@ignite.apache.org
Cc: Igor Sapego <isapego@gridgain.com>
Content-Type: multipart/alternative; boundary=001a11482a729235da0540bcf6fd
archived-at: Mon, 07 Nov 2016 21:47:01 -0000

--001a11482a729235da0540bcf6fd
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Couldn't agree more about ODBC and JDBC. We must support savepoints from
SLQ, given the DML functionality being planned for 1.8 release.

On Mon, Nov 7, 2016 at 11:23 AM, Denis Magda <dmagda@gridgain.com> wrote:

> This is how how the savepoints are supported by PostgreSQL [1], Oracle [2=
]
> and MySQL [3]. The implementation details and behavior are pretty the sam=
e
> and similar to the Yakov=E2=80=99s proposal.
>
> It worth to note that all the engines support multiple savepoints per
> transaction named uniquely and the RELEASE statement. If the one start a
> second savepoint with the name of an already existed one then the old
> savepoint will be erased/deleted.
>
> In addition it makes sense to support the feature at the level of our JDB=
C
> [4] and ODBC creating respective sub-tasks. Igor, I=E2=80=99ve googled th=
at ODBC
> supports savepoints but didn=E2=80=99t succeed at finding exact APIs. Ple=
ase assist.
>
> [1] https://www.postgresql.org/docs/8.1/static/sql-savepoint.html <
> https://www.postgresql.org/docs/8.1/static/sql-savepoint.html>
> [2] https://docs.oracle.com/cd/B19306_01/server.102/b14200/
> statements_10001.htm <https://docs.oracle.com/cd/
> B19306_01/server.102/b14200/statements_10001.htm>
> [3] http://dev.mysql.com/doc/refman/5.7/en/savepoint.html <
> http://dev.mysql.com/doc/refman/5.7/en/savepoint.html>
> [4] https://docs.oracle.com/javase/tutorial/jdbc/basics/
> transactions.html#set_roll_back_savepoints
>
> =E2=80=94
> Denis
>
> > On Nov 7, 2016, at 9:13 AM, Dmitriy Setrakyan <dsetrakyan@apache.org>
> wrote:
> >
> > Does anyone know how MySQL or PostgreSQL work with checkpoints? Do they
> > support it in a similar way?
> >
> > On Mon, Nov 7, 2016 at 5:58 AM, Yakov Zhdanov <yzhdanov@apache.org>
> wrote:
> >
> >> Alex, I think we should put consecutive checkpoints to stack thus your
> >> example should be illegal and should result to exception. And we will
> throw
> >> exception on rollback to CP if CP is not defined.
> >>
> >> --Yakov
> >>
> >> 2016-11-07 14:23 GMT+03:00 Alexey Goncharuk <alexey.goncharuk@gmail.co=
m
> >:
> >>
> >>> We also should define savepoint behavior and API rules for each of th=
e
> >>> transaction concurrency and isolation types.
> >>>
> >>> * Should rollbackToSavepoint() always release locks or clear saved
> >>> optimistic versions?
> >>> * Are forward-rollbacks allowed, e.g.
> >>>
> >>> try (Transaction tx =3D ignite.transactions().txStart()) {
> >>>    c.put(1, 1);
> >>>
> >>>    tx.savepoint("sp1");
> >>>
> >>>    c.put(2, 2);
> >>>
> >>>    tx.savepoint("sp2")
> >>>
> >>>    c.put(3, 3);
> >>>
> >>>    tx.rollbackToSavepoint("sp1");
> >>>
> >>>    c.put(4, 4);
> >>>
> >>>    tx.rollbackToSavepoint("sp2"); // Is this allowed?
> >>>
> >>>    tx.commit();
> >>> }
> >>>
> >>>
> >>> 2016-11-07 13:47 GMT+03:00 Sergey Kozlov <skozlov@gridgain.com>:
> >>>
> >>>> Hi, Yakov
> >>>>
> >>>> I suppose it's very very handy feature.
> >>>> My two cents are following:
> >>>> - number of savepoints may be more than one per transaction
> >>>> - what's about RELEASE SAVEPOINT statement?
> >>>>
> >>>>
> >>>> On Mon, Nov 7, 2016 at 11:24 AM, Yakov Zhdanov <yzhdanov@apache.org>
> >>>> wrote:
> >>>>
> >>>>> Guys,
> >>>>>
> >>>>> Let's think of implementing savepoints for Ignite transactions. For
> >> me,
> >>>>> this is not too hard to do.
> >>>>>
> >>>>> Having "savepoints" seems to be pretty handy. Please consider the
> >>>> following
> >>>>> snippet.
> >>>>>
> >>>>> ignite.transactions.;
> >>>>> INSERT INTO table1 VALUES (1);
> >>>>> SAVEPOINT my_savepoint;
> >>>>> INSERT INTO table1 VALUES (2);
> >>>>> ROLLBACK TO SAVEPOINT my_savepoint;
> >>>>> INSERT INTO table1 VALUES (3);
> >>>>> COMMIT;
> >>>>>
> >>>>> Which should result in values 1 and 3 inserted to table1.
> >>>>>
> >>>>> In Ignite, I think,  it would be like the following (preserving the
> >>> same
> >>>>> behavior as above).
> >>>>>
> >>>>> Ignite ignite =3D ....;
> >>>>> IgniteCache<Integer, Integer> c =3D ....;
> >>>>>
> >>>>> try (Transaction tx =3D ignite.transactions().txStart()) {
> >>>>>    c.put(1, 1);
> >>>>>
> >>>>>    tx.savepoint("mysavepoint");
> >>>>>
> >>>>>    c.put(2, 2);
> >>>>>
> >>>>>    tx.rollbackToSavepoint("mysavepoint");
> >>>>>
> >>>>>    c.put(3, 3);
> >>>>>
> >>>>>    tx.commit();
> >>>>> }
> >>>>>
> >>>>> As far as implementation complexity I would give 2 weeks ballpark.
> >>>>>
> >>>>> 5 days - API design and implementation for different types of
> >>>> transactions
> >>>>> - savepoint implementation for local transaction objects
> >>>>> - rollback implementation for different types of transactions - dra=
in
> >>>> local
> >>>>> tx buffers, release remote locks for pessimistic transactions, etc.
> >>>>>
> >>>>> 5 days - testing, benchmarking, fixing issues.
> >>>>>
> >>>>> Please leave your comments here. I will file a ticket after we
> >> discuss
> >>>> this
> >>>>> and put our summary to it.
> >>>>>
> >>>>> Thanks!
> >>>>>
> >>>>> --Yakov
> >>>>>
> >>>>
> >>>>
> >>>>
> >>>> --
> >>>> Sergey Kozlov
> >>>> GridGain Systems
> >>>> www.gridgain.com
> >>>>
> >>>
> >>
>
>

--001a11482a729235da0540bcf6fd--