Skip to content

Stable cs to analytics #9484

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 1 commit into from
Sep 20, 2024
Merged

Stable cs to analytics #9484

merged 1 commit into from
Sep 20, 2024

Conversation

zverevgeny
Copy link
Collaborator

No description provided.

@zverevgeny zverevgeny requested a review from a team as a code owner September 19, 2024 08:23
@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-19 08:23:48 UTC Pre-commit check for 77c5657 has started.
⚪ 2024-09-19 08:26:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-19 09:14:03 UTC Build successful.
⚪ 2024-09-19 09:14:18 UTC Tests are running...
🔴 2024-09-19 11:22:39 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76476 62990 0 9 13435 42

🔴 2024-09-19 11:23:28 UTC ydbd size 8.2 GiB changed* by +32.5 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: 89eba5c merge: 77c5657 diff diff %
ydbd size 8 792 060 368 Bytes 8 826 144 176 Bytes +32.5 MiB +0.388%
ydbd stripped size 480 119 496 Bytes 480 971 784 Bytes +832.3 KiB +0.178%

*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 GitHub Actions
Copy link

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

⚪ 2024-09-19 08:26:34 UTC Pre-commit check for 77c5657 has started.
⚪ 2024-09-19 08:29:37 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-19 08:45:35 UTC Build successful.

@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-19 08:27:25 UTC Pre-commit check for 77c5657 has started.
⚪ 2024-09-19 08:30:27 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-19 08:58:31 UTC Build successful.
⚪ 2024-09-19 08:58:46 UTC Tests are running...
🔴 2024-09-19 11:08:28 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15304 15171 0 36 51 46

🔴 2024-09-19 11:09:27 UTC ydbd size 5.6 GiB changed* by +21.0 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: ffe45cd merge: 77c5657 diff diff %
ydbd size 5 955 858 384 Bytes 5 977 869 792 Bytes +21.0 MiB +0.370%
ydbd stripped size 1 491 895 616 Bytes 1 495 405 824 Bytes +3.3 MiB +0.235%

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

@zverevgeny
Cs to stable 24 3 (ydb-platform#9268)
4c494cc 
    Co-authored-by: Semyon <yentsovsemyon@ydb.tech>
    Co-authored-by: ivanmorozov333 <ivanmorozov@ydb.tech>
    Co-authored-by: Ivan <5627721+abyss7@users.noreply.github.com>
    Co-authored-by: Artem Alekseev <fexolm@ydb.tech>
    Co-authored-by: Alexander Avdonkin <aavdonkin@yandex.ru>
    Co-authored-by: Nikita Vasilev <ns-vasilev@ydb.tech>
    Co-authored-by: Vladislav Gogov <vlad-gogov@ydb.tech>
    Conflicts:
            ydb/core/kqp/compute_actor/kqp_compute_actor.cpp
            ydb/core/kqp/compute_actor/kqp_compute_actor.h
            ydb/core/kqp/compute_actor/kqp_compute_actor_factory.h
            ydb/core/kqp/compute_actor/kqp_compute_events.h
            ydb/core/kqp/compute_actor/kqp_scan_compute_actor.cpp
            ydb/core/kqp/compute_actor/kqp_scan_compute_actor.h
            ydb/core/kqp/compute_actor/kqp_scan_fetcher_actor.cpp
            ydb/core/kqp/compute_actor/kqp_scan_fetcher_actor.h
            ydb/core/kqp/executer_actor/kqp_data_executer.cpp
            ydb/core/kqp/executer_actor/kqp_planner.cpp
            ydb/core/kqp/executer_actor/kqp_planner.h
            ydb/core/kqp/node_service/kqp_node_service.cpp
            ydb/core/kqp/ut/common/kqp_ut_common.cpp
            ydb/core/kqp/ut/olap/clickbench_ut.cpp
            ydb/core/kqp/ut/olap/kqp_olap_ut.cpp
            ydb/core/kqp/ut/tx/kqp_sink_common.h
            ydb/core/kqp/ut/tx/kqp_sink_locks_ut.cpp
            ydb/core/kqp/ut/tx/kqp_sink_mvcc_ut.cpp
            ydb/core/kqp/ut/tx/kqp_sink_tx_ut.cpp
            ydb/core/protos/feature_flags.proto
            ydb/core/protos/kqp.proto
            ydb/core/testlib/basics/feature_flags.h
            ydb/core/tx/columnshard/engines/scheme/index_info.cpp
            ydb/core/tx/columnshard/splitter/chunks.h
            ydb/library/yql/sql/v1/sql_ut.cpp
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-12
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-14
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-15
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-16
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-17
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-21
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-30
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-31
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-32
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-33
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-34
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-36
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-37
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-38
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-41
            ydb/tests/functional/clickbench/canondata/test.test_plans_column_/queries-original-plan-column-7

Fix tx tests (ydb-platform#8945) (ydb-platform#9284)

    Co-authored-by: Nikita Vasilev <ns-vasilev@ydb.tech>

fix tests
@zverevgeny zverevgeny force-pushed the stable_cs_to_analytics branch from f0f9d15 to 4c494cc Compare September 20, 2024 04:32
@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-20 04:36:10 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 04:39:15 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-20 05:18:07 UTC Build successful.

@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-20 04:36:23 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 04:39:32 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-20 05:24:06 UTC Build successful.
⚪ 2024-09-20 05:24:20 UTC Tests are running...
🔴 2024-09-20 06:13:37 UTC Test run completed, no test results found for commit 4c494cc. Please check test log.
🔴 2024-09-20 06:14:19 UTC ydbd size 5.6 GiB changed* by +21.0 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: 79bc3c2 merge: 2defca3 diff diff %
ydbd size 5 956 380 168 Bytes 5 978 394 488 Bytes +21.0 MiB +0.370%
ydbd stripped size 1 492 007 936 Bytes 1 495 515 936 Bytes +3.3 MiB +0.235%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation
⚫ 2024-09-20 06:14:21 UTC Check cancelled

@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-20 04:36:23 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 04:39:36 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-20 05:50:40 UTC Build successful.
🔴 2024-09-20 05:50:57 UTC Test run completed, no test results found for commit 4c494cc. Please check test log.
🔴 2024-09-20 05:51:39 UTC ydbd size 8.2 GiB changed* by +32.5 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: 79bc3c2 merge: 2defca3 diff diff %
ydbd size 8 792 764 112 Bytes 8 826 849 552 Bytes +32.5 MiB +0.388%
ydbd stripped size 480 150 984 Bytes 481 003 688 Bytes +832.7 KiB +0.178%

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

@zverevgeny zverevgeny added the rebase-and-check Rebase PR with the current base branch and check label Sep 20, 2024
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Sep 20, 2024
@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-20 06:18:18 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 06:21:22 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-20 07:02:11 UTC Build successful.
⚪ 2024-09-20 07:02:32 UTC Tests are running...
🔴 2024-09-20 09:13:39 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76484 63002 0 4 13435 43

🔴 2024-09-20 09:14:34 UTC ydbd size 8.2 GiB changed* by +32.5 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: 79bc3c2 merge: 2defca3 diff diff %
ydbd size 8 792 764 112 Bytes 8 826 849 552 Bytes +32.5 MiB +0.388%
ydbd stripped size 480 150 984 Bytes 481 003 688 Bytes +832.7 KiB +0.178%

*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 GitHub Actions
Copy link

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

⚪ 2024-09-20 06:18:20 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 06:21:24 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-20 06:28:13 UTC Build successful.

@github-actions GitHub Actions
Copy link

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

⚪ 2024-09-20 06:18:25 UTC Pre-commit check for 2defca3 has started.
⚪ 2024-09-20 06:21:28 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-20 06:45:58 UTC Build successful.
⚪ 2024-09-20 06:46:12 UTC Tests are running...
🔴 2024-09-20 08:56:54 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15311 15183 0 32 52 44

🔴 2024-09-20 08:57:45 UTC ydbd size 5.6 GiB changed* by +21.0 MiB, which is >= 2.0 MiB vs stable-24-3-8-analytics: Alert

ydbd size dash stable-24-3-8-analytics: 79bc3c2 merge: 2defca3 diff diff %
ydbd size 5 956 380 168 Bytes 5 978 394 488 Bytes +21.0 MiB +0.370%
ydbd stripped size 1 492 007 936 Bytes 1 495 515 936 Bytes +3.3 MiB +0.235%

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

nikvas0
nikvas0 approved these changes Sep 20, 2024
View reviewed changes
Copy link
Collaborator

@nikvas0 nikvas0 left a comment

Choose a reason for hiding this comment

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

Kqp part looks ok

@zverevgeny zverevgeny merged commit b87553f into ydb-platform:stable-24-3-8-analytics Sep 20, 2024
11 of 17 checks passed
@StekPerepolnen StekPerepolnen mentioned this pull request Oct 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikvas0 nikvas0 nikvas0 approved these changes

@ivanmorozov333 ivanmorozov333 ivanmorozov333 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.
3 participants
@zverevgeny @nikvas0 @ivanmorozov333