Skip to content

YQ-3684 passed database id into workload manager and resource pool classifiers tables #9610

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
Merged
merged 6 commits into from
Oct 9, 2024
Merged

YQ-3684 passed database id into workload manager and resource pool classifiers tables #9610

merged 6 commits into from
Oct 9, 2024

Conversation

@GrigoriyPA
Copy link
Collaborator

@GrigoriyPA GrigoriyPA commented Sep 23, 2024 •
edited
Loading

Changelog entry

Passed database id into workload manager and resource pool classifiers tables only for serverless databases; in format:
"<scheme shard id>:<path id>:<database path>"

Changelog category

  • Bugfix

Additional information

@GrigoriyPA GrigoriyPA requested a review from a team as a code owner September 23, 2024 06:50
@github-actions
Copy link

github-actions bot commented Sep 23, 2024 •
edited
Loading

⚪ 2024-09-23 06:53:35 UTC Pre-commit check for 8923bf5 has started.
⚪ 2024-09-23 06:56:09 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-23 07:02:48 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Sep 23, 2024 •
edited
Loading

⚪ 2024-09-23 06:53:36 UTC Pre-commit check for 8923bf5 has started.
⚪ 2024-09-23 06:56:13 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-23 07:20:06 UTC Build successful.
⚪ 2024-09-23 07:20:24 UTC Tests are running...
🔴 2024-09-23 09:19:54 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10102 10017 0 13 26 46

🟡 2024-09-23 09:20:39 UTC ydbd size 5.6 GiB changed* by +346.8 KiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: b87553f merge: 8923bf5 diff diff %
ydbd size 5 978 394 584 Bytes 5 978 749 728 Bytes +346.8 KiB +0.006%
ydbd stripped size 1 495 516 000 Bytes 1 495 581 696 Bytes +64.2 KiB +0.004%

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

@github-actions
Copy link

github-actions bot commented Sep 23, 2024 •
edited
Loading

⚪ 2024-09-23 06:53:36 UTC Pre-commit check for 8923bf5 has started.
⚪ 2024-09-23 06:56:16 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-23 07:36:33 UTC Build successful.
⚪ 2024-09-23 07:36:47 UTC Tests are running...
🔴 2024-09-23 09:01:49 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14401 13050 0 5 1303 43

🟡 2024-09-23 09:02:32 UTC ydbd size 8.2 GiB changed* by +683.0 KiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: b87553f merge: 8923bf5 diff diff %
ydbd size 8 826 849 584 Bytes 8 827 548 976 Bytes +683.0 KiB +0.008%
ydbd stripped size 481 003 688 Bytes 481 021 480 Bytes +17.4 KiB +0.004%

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

@GrigoriyPA GrigoriyPA changed the title YQ-3684 fixed error duplicate session id YQ-3684 passed database id into workload manager and resource pool classifiers tables Sep 26, 2024
@github-actions
Copy link

github-actions bot commented Sep 26, 2024 •
edited
Loading

⚪ 2024-09-26 15:48:43 UTC Pre-commit check for a3cb72b has started.
⚪ 2024-09-26 15:51:45 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-26 16:54:56 UTC Build successful.
⚪ 2024-09-26 16:55:11 UTC Tests are running...
🔴 2024-09-26 18:42:23 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
40101 34838 0 3 5218 42

🟡 2024-09-26 18:43:14 UTC ydbd size 8.2 GiB changed* by +1.6 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: b87553f merge: a3cb72b diff diff %
ydbd size 8 826 849 584 Bytes 8 828 550 128 Bytes +1.6 MiB +0.019%
ydbd stripped size 481 003 688 Bytes 481 061 992 Bytes +56.9 KiB +0.012%

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

@github-actions
Copy link

github-actions bot commented Sep 26, 2024 •
edited
Loading

⚪ 2024-09-26 15:48:50 UTC Pre-commit check for a3cb72b has started.
⚪ 2024-09-26 15:51:51 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-26 16:32:49 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Sep 26, 2024 •
edited
Loading

⚪ 2024-09-26 15:48:54 UTC Pre-commit check for a3cb72b has started.
⚪ 2024-09-26 15:51:46 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-26 16:33:04 UTC Build successful.
⚪ 2024-09-26 16:33:28 UTC Tests are running...
🔴 2024-09-26 18:32:33 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11543 11453 0 17 28 45

🟡 2024-09-26 18:33:18 UTC ydbd size 5.6 GiB changed* by +1.2 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: b87553f merge: a3cb72b diff diff %
ydbd size 5 978 394 584 Bytes 5 979 652 144 Bytes +1.2 MiB +0.021%
ydbd stripped size 1 495 516 000 Bytes 1 495 711 264 Bytes +190.7 KiB +0.013%

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

@GrigoriyPA GrigoriyPA force-pushed the merge-to-analytics-stable-YQ-WM-fix-error-duplicate-session-id branch from 09731f3 to 564bf8a Compare October 7, 2024 11:14
@github-actions
Copy link

github-actions bot commented Oct 7, 2024 •
edited
Loading

⚪ 2024-10-07 11:17:27 UTC Pre-commit check for 379646b has started.
⚪ 2024-10-07 11:20:29 UTC Build linux-x86_64-release-clang14 is running...
🔴 2024-10-07 11:43:57 UTC Build failed. see the build logs.

@github-actions
Copy link

github-actions bot commented Oct 7, 2024 •
edited
Loading

⚪ 2024-10-07 11:18:33 UTC Pre-commit check for 379646b has started.
⚪ 2024-10-07 11:21:27 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-10-07 11:30:35 UTC Build failed. see the build logs.
🔴 2024-10-07 11:30:36 UTC Tests run skipped.

@github-actions
Copy link

github-actions bot commented Oct 7, 2024 •
edited
Loading

⚪ 2024-10-07 11:19:48 UTC Pre-commit check for 379646b has started.
⚪ 2024-10-07 11:22:39 UTC Build linux-x86_64-release-asan is running...
🔴 2024-10-07 11:32:08 UTC Build failed. see the build logs.
🔴 2024-10-07 11:32:10 UTC Tests run skipped.

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 07:59:02 UTC Pre-commit check for e899401 has started.
⚪ 2024-10-08 08:01:53 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-10-08 08:22:02 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 07:59:08 UTC Pre-commit check for e899401 has started.
⚪ 2024-10-08 08:02:08 UTC Build linux-x86_64-release-asan is running...
🟢 2024-10-08 08:44:56 UTC Build successful.
⚪ 2024-10-08 08:45:17 UTC Tests are running...
🔴 2024-10-08 10:50:18 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11546 11455 0 21 26 44

🟡 2024-10-08 10:51:03 UTC ydbd size 5.6 GiB changed* by +1.5 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: c7bd3d0 merge: e899401 diff diff %
ydbd size 5 978 394 584 Bytes 5 980 009 592 Bytes +1.5 MiB +0.027%
ydbd stripped size 1 495 516 000 Bytes 1 495 755 968 Bytes +234.3 KiB +0.016%

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

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 08:01:53 UTC Pre-commit check for e899401 has started.
⚪ 2024-10-08 08:04:45 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-10-08 09:00:54 UTC Build successful.
⚪ 2024-10-08 09:01:10 UTC Tests are running...
🔴 2024-10-08 10:50:22 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
40103 34838 0 5 5218 42

🟡 2024-10-08 10:51:06 UTC ydbd size 8.2 GiB changed* by +1.9 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: c7bd3d0 merge: e899401 diff diff %
ydbd size 8 826 849 552 Bytes 8 828 820 664 Bytes +1.9 MiB +0.022%
ydbd stripped size 481 003 688 Bytes 481 074 472 Bytes +69.1 KiB +0.015%

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

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:17:20 UTC Pre-commit check for d024d60 has started.
⚫ 2024-10-08 12:19:14 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:17:27 UTC Pre-commit check for d024d60 has started.
⚫ 2024-10-08 12:19:22 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:18:56 UTC Pre-commit check for d024d60 has started.
⚫ 2024-10-08 12:19:15 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:20:43 UTC Pre-commit check for e56bf48 has started.
⚪ 2024-10-08 12:23:32 UTC Build linux-x86_64-release-asan is running...
🟢 2024-10-08 12:46:11 UTC Build successful.
⚪ 2024-10-08 12:46:32 UTC Tests are running...
🔴 2024-10-08 14:52:20 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11547 11456 0 20 28 43

🟡 2024-10-08 14:52:58 UTC ydbd size 5.6 GiB changed* by +1.5 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: c7bd3d0 merge: e56bf48 diff diff %
ydbd size 5 978 394 584 Bytes 5 980 009 592 Bytes +1.5 MiB +0.027%
ydbd stripped size 1 495 516 000 Bytes 1 495 755 968 Bytes +234.3 KiB +0.016%

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

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:20:45 UTC Pre-commit check for e56bf48 has started.
⚪ 2024-10-08 12:23:41 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-10-08 12:30:06 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Oct 8, 2024 •
edited
Loading

⚪ 2024-10-08 12:20:52 UTC Pre-commit check for e56bf48 has started.
⚪ 2024-10-08 12:23:44 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-10-08 13:02:09 UTC Build successful.
⚪ 2024-10-08 13:02:23 UTC Tests are running...
🔴 2024-10-08 14:52:23 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
40104 34836 0 2 5221 45

🟡 2024-10-08 14:53:18 UTC ydbd size 8.2 GiB changed* by +1.9 MiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: c7bd3d0 merge: e56bf48 diff diff %
ydbd size 8 826 849 552 Bytes 8 828 820 664 Bytes +1.9 MiB +0.022%
ydbd stripped size 481 003 688 Bytes 481 074 472 Bytes +69.1 KiB +0.015%

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

@GrigoriyPA GrigoriyPA merged commit 245dfa3 into ydb-platform:stable-24-3-8-analytics Oct 9, 2024
4 of 6 checks passed
@GrigoriyPA GrigoriyPA deleted the merge-to-analytics-stable-YQ-WM-fix-error-duplicate-session-id branch October 9, 2024 09:53
@StekPerepolnen StekPerepolnen mentioned this pull request Oct 29, 2024
Closed
This was referenced Jun 3, 2025
Closed
Closed
@StekPerepolnen StekPerepolnen mentioned this pull request Jul 28, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fomichev3000 fomichev3000 fomichev3000 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@GrigoriyPA @fomichev3000