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

Transaction order regression caused by 5197 #5416

Closed
shiveshr opened this issue Dec 10, 2020 · 0 comments · Fixed by #5417
Closed

Transaction order regression caused by 5197 #5416

shiveshr opened this issue Dec 10, 2020 · 0 comments · Fixed by #5417

Comments

@shiveshr
Copy link
Contributor

Problem description
With #5197 we regressed the transaction ordering if the number of transactions in committing state are more than the limit (100) as we do not honor the order among them and end up picking 100 of them and then applying order on the chosen list.
the list should be chosen in order itself

Problem location
PersistentStreamBase

Suggestions for an improvement
Fix the bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant