Skip to content

Fix bulk upsert writing to freshly indexed table due to a race KIKIMR-20765 #1160

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

Merged

Conversation

snaury
Copy link
Member

@snaury snaury commented Jan 19, 2024

Changelog entry

Fix bulk upsert writing to freshly indexed table due to a race

Changelog category

  • Bugfix

Additional information

It was possible for bulk upsert to send datashard request based on a table schema before it had global indexes added. Consequently bulk upsert would succeed and insert new data without updating global indexes, causing inconsistencies between data and indexes.

@snaury snaury requested a review from gridnevvvit January 19, 2024 12:51
Copy link

github-actions bot commented Jan 19, 2024

2024-01-19 12:54:04 UTC Pre-commit check for 79e7d09 has started.
2024-01-19 12:54:06 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-19 13:29:20 UTC Build successful.
2024-01-19 13:29:33 UTC Tests are running...
🔴 2024-01-19 14:50:45 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
59993 50740 0 4 9235 14

@snaury snaury marked this pull request as ready for review January 19, 2024 12:55
Copy link

github-actions bot commented Jan 19, 2024

2024-01-19 12:56:40 UTC Pre-commit check for 79e7d09 has started.
2024-01-19 12:56:42 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-19 13:37:00 UTC Build successful.
2024-01-19 13:37:15 UTC Tests are running...
🔴 2024-01-19 15:14:52 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15887 15789 0 16 72 10

@snaury snaury requested a review from CyberROFL January 19, 2024 14:00
@snaury snaury merged commit d6ad10e into ydb-platform:main Jan 19, 2024
@snaury snaury deleted the KIKIMR-20765-bulk-upsert-schema-race branch January 19, 2024 15:08
@niksaveliev niksaveliev mentioned this pull request Jan 29, 2024
@vitstn vitstn mentioned this pull request Jan 29, 2024
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants