Skip to content

Relative time sensors and time series backport #10640

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

Hor911
Copy link
Collaborator

@Hor911 Hor911 commented Oct 19, 2024

Backport of new-style stat reporting. Main reason - use of relative time sensors instead of absolute ones to reduce size of JSON and enable time series (YQ only). Timeline visualizer doesn't support old absolute time, and it looks like this:

image

New beaviour is as following:

image

@Hor911 Hor911 requested a review from a team as a code owner October 19, 2024 07:30
Copy link

github-actions bot commented Oct 19, 2024

2024-10-19 07:34:06 UTC Pre-commit check for 89ae7bc has started.
2024-10-19 07:37:18 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-10-19 08:22:56 UTC Build successful.

Copy link

github-actions bot commented Oct 19, 2024

2024-10-19 07:34:09 UTC Pre-commit check for 89ae7bc has started.
2024-10-19 07:37:15 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-10-19 08:43:47 UTC Build successful.
2024-10-19 08:44:03 UTC Tests are running...
🔴 2024-10-19 10:58:12 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
73460 59970 0 16 13432 42

🟡 2024-10-19 10:59:05 UTC ydbd size 8.2 GiB changed* by +199.2 KiB, which is >= 100.0 KiB vs stable-24-3-10-analytics: Warning

ydbd size dash stable-24-3-10-analytics: 7e2dcef merge: 89ae7bc diff diff %
ydbd size 8 846 299 184 Bytes 8 846 503 200 Bytes +199.2 KiB +0.002%
ydbd stripped size 483 049 544 Bytes 483 060 104 Bytes +10.3 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

Copy link

github-actions bot commented Oct 19, 2024

2024-10-19 07:34:16 UTC Pre-commit check for 89ae7bc has started.
2024-10-19 07:37:13 UTC Build linux-x86_64-release-asan is running...
🟢 2024-10-19 08:28:14 UTC Build successful.
2024-10-19 08:28:31 UTC Tests are running...
🔴 2024-10-19 10:38:16 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
12374 12250 0 29 51 44

🟢 2024-10-19 10:39:02 UTC ydbd size 5.6 GiB changed* by +80.8 KiB, which is < 100.0 KiB vs stable-24-3-10-analytics: OK

ydbd size dash stable-24-3-10-analytics: 7e2dcef merge: 89ae7bc diff diff %
ydbd size 5 982 312 904 Bytes 5 982 395 624 Bytes +80.8 KiB +0.001%
ydbd stripped size 1 499 869 056 Bytes 1 499 892 768 Bytes +23.2 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

@Hor911 Hor911 changed the title Timeline and time series better support Relative time sensors and time series backport Oct 19, 2024
@Hor911 Hor911 enabled auto-merge (squash) October 19, 2024 10:19
@Hor911 Hor911 merged commit 75503d5 into ydb-platform:stable-24-3-10-analytics Oct 19, 2024
8 of 12 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.

2 participants