Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WiP: first test to try to resolve #507 #508

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

FCO
Copy link
Owner

@FCO FCO commented Aug 29, 2021

If red-do :transaction's block returns a Awaitable, it creates
a new Promise that will handle the commit/rollback and does not
let the sync code to call commit/rollback.

Probable next steps are:

  • extract that logic to a transaction manager
  • on transaction manager add a transaction stack and methods
    • queue-begin
    • unqueue-commit
    • break-queue-rollback
      (or something like that)

If `red-do :transaction`'s block returns a Awaitable, it creates
a new Promise that will handle the commit/rollback and does not
let the sync code to call commit/rollback.

Probable next steps are:
- extract that logic to a transaction manager
- on transaction manager add a transaction stack and methods
	- queue-begin
	- unqueue-commit
	- break-queue-rollback
(or something like that)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant