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.

Sign up for GitHub

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

Jump to bottom

[Dist-txn]: Clean up provisional records of aborted transactions during compactions #289

Closed
mbautin opened this issue May 15, 2018 · 1 comment
Closed

[Dist-txn]: Clean up provisional records of aborted transactions during compactions #289

mbautin opened this issue May 15, 2018 · 1 comment
Assignees
@rven1
Labels
kind/bug This issue is a bug
Projects
Distributed transactions

Comments

@mbautin
Copy link
Collaborator

mbautin commented May 15, 2018

Could test this with a workload that starts and aborts a lot of transactions.
@mbautin mbautin added the kind/bug This issue is a bug label May 15, 2018
@mbautin mbautin added this to To Do in Distributed transactions via automation May 15, 2018
@mbautin mbautin changed the title Clean up provisional records of aborted transactions during compactions [Dist-txn]: Clean up provisional records of aborted transactions during compactions May 15, 2018
@robertpang robertpang moved this from To Do to In progress in Distributed transactions May 30, 2018
yugabyte-ci pushed a commit that referenced this issue Sep 5, 2018
@rven1
[#289] Cleanup intents for aborted transactions in the compaction path
cdbf60c 
Summary:
This diff implements the compaction path for
cleaning the intents for aborted transactions.

Test Plan: QLTransactionTest

Reviewers: mikhail, robert, sergei

Reviewed By: sergei

Subscribers: kannan, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D5294
@rven1
Copy link
Contributor

rven1 commented Sep 7, 2018

Have committed changes to clean up intents for aborted transactions in
a) abort from client
b) transactions aborted because of conflicts
c) expired transactions because of failure to send heartbeat from client
d) in read path when an aborted transaction is encountered
e) in compaction path, when an aborted transaction is encountered.

@rven1 rven1 closed this as completed Sep 7, 2018
Distributed transactions automation moved this from In progress to Done Sep 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rven1 rven1

Labels
kind/bug This issue is a bug
Projects
Distributed transactions
  
Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mbautin @rven1