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

Extra rows in the asynchronous index #4034

Closed
CyberROFL opened this issue Apr 23, 2024 · 0 comments · Fixed by #4055, #4087, #4105, #4106 or #4177
Closed

Extra rows in the asynchronous index #4034

CyberROFL opened this issue Apr 23, 2024 · 0 comments · Fixed by #4055, #4087, #4105, #4106 or #4177
Assignees
@CyberROFL
Labels
bug Something isn't working

Comments

@CyberROFL
Copy link
Member

CyberROFL commented Apr 23, 2024 •
edited
Loading

Race between Resolve & SendRecords:

  • Enqueue records R1, R2.
  • Request their bodies.
  • Receive R1, send R1 to partition P1.
  • Error: WRONG_SHARD_STATE.
  • Re-enqueue R1, resolve partitions.
  • Resolve done: new partitions are P2, P3.
  • Receive R2, send to P2.
  • Receive R1, send to P2 — ignore.
@CyberROFL CyberROFL self-assigned this Apr 23, 2024
This was linked to pull requests Apr 23, 2024
Additional tests with reboots #4055
Merged
Fix reordering of change records #4087
Merged
@CyberROFL CyberROFL changed the title Extra rows in the covering asynchronous index Extra rows in the asynchronous index Apr 24, 2024
This was linked to pull requests Apr 25, 2024
24-1: Fix reordering of change records (#4087) #4105
Merged
24-1-14-hotfix: Fix reordering of change records (#4087) #4106
Merged
@CyberROFL CyberROFL added the bug Something isn't working label Apr 25, 2024
@CyberROFL CyberROFL linked a pull request Apr 25, 2024 that will close this issue
Debug printing of change records #4118
Merged
@CyberROFL CyberROFL removed a link to a pull request Apr 25, 2024
Debug printing of change records #4118
Merged
@CyberROFL CyberROFL linked a pull request Apr 27, 2024 that will close this issue
Race test between Resolve() & SendRecords() #4177
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CyberROFL CyberROFL

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@CyberROFL