Skip to content

Merge to 24-4: Node TNodeLocation processing to NodeWarden in order to reduce CPU consumption (#12776) #17036

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

Open
wants to merge 1 commit into
base: stable-24-4
Choose a base branch
from

Conversation

kruall
Copy link
Collaborator

@kruall kruall commented Apr 10, 2025

Changelog entry

...

Changelog category

  • Not for changelog (changelog entry is not required)

Description for reviewers

Node TNodeLocation processing to NodeWarden in order to reduce CPU consumption (#12776)

@kruall kruall requested a review from a team as a code owner April 10, 2025 07:46
@kruall kruall added the area/blobstorage BlobStorage issues label Apr 10, 2025
Copy link

github-actions bot commented Apr 10, 2025

2025-04-10 07:48:59 UTC Pre-commit check for c532fe7 has started.
2025-04-10 07:51:45 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-04-10 08:37:48 UTC Build successful.
2025-04-10 08:38:02 UTC Tests are running...
🔴 2025-04-10 10:08:50 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16062 14693 0 22 1302 45

🟢 2025-04-10 10:09:45 UTC ydbd size 8.3 GiB changed* by +81.6 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: 4e8794d merge: c532fe7 diff diff %
ydbd size 8 898 210 744 Bytes 8 898 294 264 Bytes +81.6 KiB +0.001%
ydbd stripped size 485 016 168 Bytes 485 015 528 Bytes -640 Bytes -0.000%

*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 Apr 10, 2025

2025-04-10 07:49:24 UTC Pre-commit check for c532fe7 has started.
2025-04-10 07:52:12 UTC Build linux-x86_64-release-asan is running...
🟢 2025-04-10 08:19:11 UTC Build successful.
2025-04-10 08:19:27 UTC Tests are running...
🔴 2025-04-10 10:33:05 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11619 11510 0 36 28 45

🟢 2025-04-10 10:34:00 UTC ydbd size 5.6 GiB changed* by +82.7 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: 4e8794d merge: c532fe7 diff diff %
ydbd size 6 027 606 728 Bytes 6 027 691 424 Bytes +82.7 KiB +0.001%
ydbd stripped size 1 508 434 976 Bytes 1 508 444 256 Bytes +9.1 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

fomichev3000
fomichev3000 previously approved these changes Apr 10, 2025
Copy link

github-actions bot commented Apr 10, 2025

2025-04-10 12:12:22 UTC Pre-commit check for beaee05 has started.
2025-04-10 12:15:50 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2025-04-10 12:56:44 UTC Build successful.
2025-04-10 12:57:02 UTC Tests are running...
🔴 2025-04-10 14:38:54 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16064 14691 0 22 1304 47

🟢 2025-04-10 14:39:55 UTC ydbd size 8.3 GiB changed* by +87.2 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: 4e8794d merge: beaee05 diff diff %
ydbd size 8 898 210 744 Bytes 8 898 300 056 Bytes +87.2 KiB +0.001%
ydbd stripped size 485 016 168 Bytes 485 017 192 Bytes +1.0 KiB +0.000%

*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 Apr 10, 2025

2025-04-10 12:13:17 UTC Pre-commit check for beaee05 has started.
2025-04-10 12:16:04 UTC Build linux-x86_64-release-asan is running...
🟢 2025-04-10 12:38:55 UTC Build successful.
2025-04-10 12:39:14 UTC Tests are running...
🔴 2025-04-10 14:52:13 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11617 11512 0 34 29 42

🟢 2025-04-10 14:53:02 UTC ydbd size 5.6 GiB changed* by +84.9 KiB, which is < 100.0 KiB vs stable-24-4: OK

ydbd size dash stable-24-4: 4e8794d merge: beaee05 diff diff %
ydbd size 6 027 606 728 Bytes 6 027 693 648 Bytes +84.9 KiB +0.001%
ydbd stripped size 1 508 434 976 Bytes 1 508 444 448 Bytes +9.2 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/blobstorage BlobStorage issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants