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

stability: Coalesce duplicate (sync) write intent errors #8692

Closed
bdarnell opened this issue Aug 19, 2016 · 3 comments
Closed

stability: Coalesce duplicate (sync) write intent errors #8692

bdarnell opened this issue Aug 19, 2016 · 3 comments

Comments

@bdarnell
Copy link
Contributor

When many consistent reads hit the same write intent, they will each try to push the transaction and resolve the intent. We coalesce these operations if the read was inconsistent (so the resolution is performed asynchronously); we should do the same for the synchronous path used by consistent reads. It's more subtle because these pushes are not exact duplicates for each other: a higher-priority transaction could win the push while an in-progress lower-priority one could fail. We at least want to block all pushes that are less likely to succeed than the one in progress; I'm not sure whether it makes sense to allow higher-priority pushes to proceed or wait for the first one to finish first.

On the beta cluster nodes are currently running out of memory due to an accumulation of these PushTxn attempts (that so many pushes are attempted but none are succeeding is a separate issue).

@tbg
Copy link
Member

tbg commented Aug 22, 2016

see also #7503.

@tbg
Copy link
Member

tbg commented Aug 22, 2016

see also #627.

@tbg
Copy link
Member

tbg commented Sep 26, 2016

Closing in favor of #627.

@tbg tbg closed this as completed Sep 26, 2016
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

No branches or pull requests

2 participants