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

Support sequences backup with current value #3089

Merged
merged 10 commits into from
Mar 29, 2024

Conversation

shnikd
Copy link
Collaborator

@shnikd shnikd commented Mar 22, 2024

No description provided.

@shnikd shnikd requested a review from ijon March 22, 2024 12:57
Copy link

github-actions bot commented Mar 22, 2024

2024-03-22 13:00:29 UTC Pre-commit check for 1ea186f has started.
2024-03-22 13:00:30 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-22 13:34:55 UTC Build successful.

Copy link

github-actions bot commented Mar 22, 2024

2024-03-22 13:08:50 UTC Pre-commit check for 1ea186f has started.
2024-03-22 13:08:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-22 13:45:56 UTC Build successful.
2024-03-22 13:46:24 UTC Tests are running...
🔴 2024-03-22 15:42:54 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
69021 57912 0 8 11023 78

Copy link

github-actions bot commented Mar 22, 2024

2024-03-22 13:14:51 UTC Pre-commit check for 1ea186f has started.
2024-03-22 13:14:52 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-22 13:52:53 UTC Build successful.
2024-03-22 13:53:07 UTC Tests are running...
🔴 2024-03-22 16:01:20 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15149 14500 0 79 536 34

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 08:17:06 UTC Pre-commit check for ab34b07 has started.
2024-03-25 08:17:08 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-25 08:57:19 UTC Build successful.
2024-03-25 08:57:34 UTC Tests are running...
🔴 2024-03-25 09:43:20 UTC Test run completed, no test results found for commit bc90b90. Please check build logs.
2024-03-25 09:43:23 UTC Check cancelled

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 08:17:10 UTC Pre-commit check for ab34b07 has started.
2024-03-25 08:17:13 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-25 08:55:21 UTC Build successful.

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 08:19:22 UTC Pre-commit check for ab34b07 has started.
2024-03-25 08:19:24 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-25 08:57:57 UTC Build successful.
2024-03-25 08:58:14 UTC Tests are running...
🔴 2024-03-25 09:43:19 UTC Test run completed, no test results found for commit bc90b90. Please check build logs.
2024-03-25 09:43:23 UTC Check cancelled

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 09:44:38 UTC Pre-commit check for f90c9a3 has started.
2024-03-25 09:44:39 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-25 09:47:46 UTC Build successful.
2024-03-25 09:47:57 UTC Tests are running...
🔴 2024-03-25 10:37:18 UTC Test run completed, no test results found for commit 82d5b85. Please check build logs.
2024-03-25 10:37:21 UTC Check cancelled

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 09:44:38 UTC Pre-commit check for f90c9a3 has started.
2024-03-25 09:44:40 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-25 09:47:19 UTC Build successful.
2024-03-25 09:47:33 UTC Tests are running...
🔴 2024-03-25 10:37:18 UTC Test run completed, no test results found for commit 82d5b85. Please check build logs.
2024-03-25 10:37:21 UTC Check cancelled

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 09:46:46 UTC Pre-commit check for f90c9a3 has started.
2024-03-25 09:46:49 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-25 09:50:16 UTC Build successful.

@shnikd shnikd requested a review from ijon March 25, 2024 09:48
Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 10:39:14 UTC Pre-commit check for 0de43f2 has started.
2024-03-25 10:39:20 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-25 10:41:50 UTC Build successful.
2024-03-25 10:42:05 UTC Tests are running...
🔴 2024-03-25 11:29:55 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14413 14124 0 42 220 27

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 10:39:28 UTC Pre-commit check for 0de43f2 has started.
2024-03-25 10:39:30 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-25 10:41:44 UTC Build successful.

Copy link

github-actions bot commented Mar 25, 2024

2024-03-25 10:41:04 UTC Pre-commit check for 0de43f2 has started.
2024-03-25 10:41:06 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-25 10:43:28 UTC Build successful.
2024-03-25 10:43:42 UTC Tests are running...
🔴 2024-03-25 11:40:49 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
69044 57931 0 9 11031 73

ydb/core/protos/tx_sequenceshard.proto Outdated Show resolved Hide resolved
ydb/public/api/protos/ydb_table.proto Outdated Show resolved Hide resolved
ydb/core/protos/flat_scheme_op.proto Outdated Show resolved Hide resolved
ydb/core/protos/tx_sequenceshard.proto Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 12:27:55 UTC Pre-commit check for 1706b70 has started.
2024-03-26 12:27:57 UTC Build linux-x86_64-release-clang14 is running...
🔴 2024-03-26 13:04:12 UTC Build failed. see the build logs.

Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 12:28:03 UTC Pre-commit check for 1706b70 has started.
2024-03-26 12:28:05 UTC Build linux-x86_64-release-asan is running...
🔴 2024-03-26 13:07:52 UTC Build failed. see the build logs.
🔴 2024-03-26 13:07:53 UTC Tests run skipped.

Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 12:28:26 UTC Pre-commit check for 1706b70 has started.
2024-03-26 12:28:29 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-03-26 13:08:56 UTC Build failed. see the build logs.
🔴 2024-03-26 13:08:58 UTC Tests run skipped.

Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 14:00:26 UTC Pre-commit check for f28e33e has started.
2024-03-26 14:00:28 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-26 14:02:11 UTC Build successful.

Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 14:01:08 UTC Pre-commit check for f28e33e has started.
2024-03-26 14:01:10 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-26 14:03:16 UTC Build successful.
2024-03-26 14:05:10 UTC Tests are running...
🔴 2024-03-26 15:56:27 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
69048 57944 0 4 11031 69

Copy link

github-actions bot commented Mar 26, 2024

2024-03-26 14:01:10 UTC Pre-commit check for f28e33e has started.
2024-03-26 14:01:13 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-26 14:03:08 UTC Build successful.
2024-03-26 14:03:29 UTC Tests are running...
🔴 2024-03-26 15:36:34 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14419 14201 0 36 155 27

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:01:49 UTC Pre-commit check for f19d584 has started.
2024-03-27 01:01:52 UTC Build linux-x86_64-release-clang14 is running...
2024-03-27 01:10:07 UTC Check cancelled

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:01:59 UTC Pre-commit check for f19d584 has started.
2024-03-27 01:02:01 UTC Build linux-x86_64-release-asan is running...
2024-03-27 01:10:06 UTC Check cancelled

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:04:21 UTC Pre-commit check for f19d584 has started.
2024-03-27 01:04:24 UTC Build linux-x86_64-relwithdebinfo is running...
2024-03-27 01:10:07 UTC Check cancelled

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:11:15 UTC Pre-commit check for f996da0 has started.
2024-03-27 01:11:16 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-27 01:21:34 UTC Build successful.
2024-03-27 01:23:34 UTC Tests are running...
🔴 2024-03-27 03:00:53 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14076 13784 0 52 212 28

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:11:15 UTC Pre-commit check for f996da0 has started.
2024-03-27 01:11:16 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-27 01:24:09 UTC Build successful.
2024-03-27 01:26:04 UTC Tests are running...
🔴 2024-03-27 03:12:22 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68741 57614 0 9 11050 68

Copy link

github-actions bot commented Mar 27, 2024

2024-03-27 01:13:34 UTC Pre-commit check for f996da0 has started.
2024-03-27 01:13:38 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-03-27 01:23:27 UTC Build successful.

Copy link
Member

@snaury snaury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Думаю LGTM. Из совсем мелкого меня смущает, что ReturnSetVal будет возвращать не текущее значение sequence'а, а какое-то там старое, которое делали во время его создания, но это не публичное апи и потому не важно. В будущем кому-нибудь нужно будет сделать возможно получения правильного текущего set_val в публичном апи, чтобы правильно бекапить таблицы с sequence'ами можно было не только на сервере, но и локально в файлики, но точно не в этом pr.

@shnikd shnikd merged commit 3212f32 into ydb-platform:main Mar 29, 2024
3 of 5 checks passed
@shnikd shnikd linked an issue Mar 29, 2024 that may be closed by this pull request
@shnikd shnikd mentioned this pull request Apr 11, 2024
shnikd added a commit to shnikd/ydb that referenced this pull request May 30, 2024
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

Successfully merging this pull request may close these issues.

support backup of the sequences
3 participants