Add savepoint API

[fantix]

Please see tests for use case.

• [ ] Add docs

Fixes #407

[1st1]

We should allow skipping name argument and generate something unique (e.g. uuid). Actually while we have variables in python (I mean sp_name = conn.declare_savepoint() ) names are useless, will only provoke unexpected conflicts.

+1

Do we need declare_ part? x = tx.savepoint() is nice, no? (we already do that with transactions by skipping start part)

Yes, please, it should be tx.savepoint() with an optional name argument (although I'd not even have the name argument at all, why would you want it?)

[fantix]

Pushed a fix to rename declare_savepoint() to savepoint() and dropped user-defined names (always use s + uuid instead).

(Context API) can be added later, but is pretty uncontroversial I think.

Let's add later 😏

[jsimonlane]

Hi all, any updates on this?

[fantix]

Hi all, any updates on this?

Sorry, not yet. But we'll put time into the Python binding very soon!