Skip to content

Stable-24-4: initial commits #14402

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
merged 23 commits into from
Feb 11, 2025

Conversation

maximyurchuk
Copy link
Collaborator

@maximyurchuk maximyurchuk commented Feb 10, 2025

KIKIMR-21727

@maximyurchuk maximyurchuk requested a review from a team as a code owner February 10, 2025 18:33
Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:40:12 UTC Pre-commit check for c11ccf0 has started.
2025-02-10 18:43:12 UTC Build linux-x86_64-release-asan is running...
2025-02-10 18:45:20 UTC Check cancelled

Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:40:17 UTC Pre-commit check for c11ccf0 has started.
2025-02-10 18:43:22 UTC Build linux-x86_64-relwithdebinfo is running...
2025-02-10 18:45:02 UTC Check cancelled

Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:48:07 UTC Pre-commit check for 25746c5 has started.
2025-02-10 18:51:07 UTC Build linux-x86_64-relwithdebinfo is running...
2025-02-10 18:54:04 UTC Check cancelled

Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:50:27 UTC Pre-commit check for 25746c5 has started.
2025-02-10 18:53:36 UTC Build linux-x86_64-release-asan is running...
2025-02-10 18:54:41 UTC Check cancelled

Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:58:35 UTC Pre-commit check for b24e0e4 has started.
2025-02-10 19:01:36 UTC Build linux-x86_64-release-asan is running...
🟢 2025-02-10 19:53:16 UTC Build successful.
2025-02-10 19:53:31 UTC Tests are running...
🔴 2025-02-10 22:07:07 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15787 15645 0 49 51 42

🟡 2025-02-10 22:08:05 UTC ydbd size 5.6 GiB changed* by +117.7 KiB, which is >= 100.0 KiB vs stable-24-4: Warning

ydbd size dash stable-24-4: a254002 merge: b24e0e4 diff diff %
ydbd size 6 002 741 216 Bytes 6 002 861 712 Bytes +117.7 KiB +0.002%
ydbd stripped size 1 504 236 640 Bytes 1 504 258 592 Bytes +21.4 KiB +0.001%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Feb 10, 2025

2025-02-10 18:58:41 UTC Pre-commit check for b24e0e4 has started.
2025-02-10 19:01:41 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-02-10 20:08:20 UTC Build successful.
2025-02-10 20:08:33 UTC Tests are running...
🔴 2025-02-10 22:18:42 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76923 63425 0 26 13434 38

🟡 2025-02-10 22:19:36 UTC ydbd size 8.3 GiB changed* by +850.8 KiB, which is >= 100.0 KiB vs stable-24-4: Warning

ydbd size dash stable-24-4: a254002 merge: b24e0e4 diff diff %
ydbd size 8 874 098 288 Bytes 8 874 969 480 Bytes +850.8 KiB +0.010%
ydbd stripped size 483 816 072 Bytes 484 304 872 Bytes +477.3 KiB +0.101%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@maximyurchuk maximyurchuk changed the title Stable-24-4 Stable-24-4: initial commits Feb 10, 2025
@jepett0 jepett0 self-requested a review February 11, 2025 07:27
jepett0
jepett0 previously approved these changes Feb 11, 2025
optional bool EnableInsertWriteIdSpecialColumnCompatibility = 147 [default = false];
optional bool EnableDriveSerialsDiscovery = 152 [default = false];
optional bool EnableScaleRecommender = 157 [default = false];
optional bool EnableScaleRecommender = 157 [default = true];
Copy link
Member

Choose a reason for hiding this comment

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

Не готов включать по дефолту, но будет хорошо, если поучаствует в тестировании на совместимость

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Откачу

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

(откатил)

Copy link

github-actions bot commented Feb 11, 2025

2025-02-11 08:50:55 UTC Pre-commit check for 5f17198 has started.
2025-02-11 08:54:13 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-02-11 10:07:06 UTC Build successful.
2025-02-11 10:07:21 UTC Tests are running...
🔴 2025-02-11 12:27:56 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76925 63429 0 27 13432 37

🟡 2025-02-11 12:29:12 UTC ydbd size 8.3 GiB changed* by +224.5 KiB, which is >= 100.0 KiB vs stable-24-4: Warning

ydbd size dash stable-24-4: a254002 merge: 5f17198 diff diff %
ydbd size 8 874 098 288 Bytes 8 874 328 184 Bytes +224.5 KiB +0.003%
ydbd stripped size 483 816 072 Bytes 483 822 056 Bytes +5.8 KiB +0.001%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Feb 11, 2025

2025-02-11 08:54:11 UTC Pre-commit check for 5f17198 has started.
2025-02-11 08:57:21 UTC Build linux-x86_64-release-asan is running...
🟢 2025-02-11 09:46:45 UTC Build successful.
2025-02-11 09:47:02 UTC Tests are running...
🔴 2025-02-11 12:16:53 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15791 15639 0 57 55 40

🟡 2025-02-11 12:17:53 UTC ydbd size 5.6 GiB changed* by +135.2 KiB, which is >= 100.0 KiB vs stable-24-4: Warning

ydbd size dash stable-24-4: a254002 merge: 5f17198 diff diff %
ydbd size 6 002 741 216 Bytes 6 002 879 704 Bytes +135.2 KiB +0.002%
ydbd stripped size 1 504 236 640 Bytes 1 504 263 776 Bytes +26.5 KiB +0.002%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@jepett0 jepett0 self-requested a review February 11, 2025 09:54
Copy link
Collaborator

@serbel324 serbel324 left a comment

Choose a reason for hiding this comment

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

Надо будет отдельным PR оторвать совместимость со старыми NBS

@gridnevvvit gridnevvvit self-requested a review February 11, 2025 11:09
@zverevgeny
Copy link
Collaborator

ок на включение: EnableImmediateWritingOnBulkUpsert = true

@maximyurchuk
Copy link
Collaborator Author

maximyurchuk commented Feb 11, 2025

ок на включение: EnableImmediateWritingOnBulkUpsert = true

И еще не-повышение SSA_RUNTIME_VERSION
@zverevgeny

@zverevgeny
Copy link
Collaborator

ок на включение: EnableImmediateWritingOnBulkUpsert = true

И еще не-повышение SSA_RUNTIME_VERSION @zverevgeny

ок

@maximyurchuk maximyurchuk merged commit 80dbaee into ydb-platform:stable-24-4 Feb 11, 2025
6 of 10 checks passed
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.