Skip to content

YQL-17542 move TaskRunner to TDqSyncComputeActorBase#1666

Merged
zverevgeny merged 1 commit intoydb-platform:mainfrom
zverevgeny:YQL-17542_move_task_runner_to_sync_actor
Feb 8, 2024
Merged

YQL-17542 move TaskRunner to TDqSyncComputeActorBase#1666
zverevgeny merged 1 commit intoydb-platform:mainfrom
zverevgeny:YQL-17542_move_task_runner_to_sync_actor

Conversation

@zverevgeny
Copy link
Collaborator

@zverevgeny zverevgeny commented Feb 7, 2024 •
edited
Loading

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

pr over contrib: https://a.yandex-team.ru/review/5301910/details

@zverevgeny zverevgeny requested a review from a team as a code owner February 7, 2024 12:07
@github-actions
Copy link

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

⚪ 2024-02-07 12:09:41 UTC Pre-commit check for ea7f387 has started.
⚪ 2024-02-07 12:09:43 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-02-07 12:14:50 UTC Build failed. see the build logs.
⚪ 2024-02-07 12:15:02 UTC Tests are running...
🔴 2024-02-07 12:38:08 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
36217 28364 0 303 7539 11

@github-actions
Copy link

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

⚪ 2024-02-07 12:11:10 UTC Pre-commit check for ea7f387 has started.
⚪ 2024-02-07 12:11:12 UTC Build linux-x86_64-release-asan is running...
🔴 2024-02-07 12:16:40 UTC Build failed. see the build logs.
⚪ 2024-02-07 12:16:52 UTC Tests are running...
🔴 2024-02-07 12:31:42 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5981 5699 0 266 3 13

@zverevgeny zverevgeny force-pushed the YQL-17542_move_task_runner_to_sync_actor branch from d613e11 to c602299 Compare February 7, 2024 13:04
@github-actions
Copy link

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

⚪ 2024-02-07 13:32:17 UTC Pre-commit check for 93ff4cb has started.
⚪ 2024-02-07 13:32:19 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-07 13:34:09 UTC Build successful.
⚪ 2024-02-07 13:34:22 UTC Tests are running...
🔴 2024-02-07 15:08:48 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67228 56362 0 1 10818 47

@github-actions
Copy link

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

⚪ 2024-02-07 13:40:10 UTC Pre-commit check for 93ff4cb has started.
⚪ 2024-02-07 13:40:11 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-07 13:42:07 UTC Build successful.
⚪ 2024-02-07 13:42:16 UTC Tests are running...
🔴 2024-02-07 15:18:36 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14576 14458 0 14 72 32

@zverevgeny zverevgeny requested a review from Darych February 7, 2024 14:12
@zverevgeny zverevgeny force-pushed the YQL-17542_move_task_runner_to_sync_actor branch from c602299 to 2640436 Compare February 7, 2024 18:12
@github-actions
Copy link

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

⚪ 2024-02-07 18:14:47 UTC Pre-commit check for 9713ba4 has started.
⚪ 2024-02-07 18:14:49 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-07 18:16:53 UTC Build successful.
⚪ 2024-02-07 18:17:09 UTC Tests are running...
🔴 2024-02-07 19:53:18 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67240 56388 0 3 10820 29

@github-actions
Copy link

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

⚪ 2024-02-07 18:15:32 UTC Pre-commit check for 9713ba4 has started.
⚪ 2024-02-07 18:15:34 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-07 18:17:42 UTC Build successful.
⚪ 2024-02-07 18:17:58 UTC Tests are running...
🔴 2024-02-07 19:54:01 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14584 14481 0 12 57 34

@github-actions
Copy link

github-actions bot commented Feb 7, 2024

✅ Documentation build

Revision built successfully
Revision preview link

@zverevgeny zverevgeny force-pushed the YQL-17542_move_task_runner_to_sync_actor branch from 2640436 to 770e322 Compare February 8, 2024 03:59
@github-actions
Copy link

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

⚪ 2024-02-08 04:00:50 UTC Pre-commit check for ea12af9 has started.
⚪ 2024-02-08 04:00:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-08 04:02:06 UTC Build successful.
⚪ 2024-02-08 04:02:22 UTC Tests are running...
🔴 2024-02-08 04:26:01 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67240 56388 0 3 10820 29

@github-actions
Copy link

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

⚪ 2024-02-08 04:03:08 UTC Pre-commit check for ea12af9 has started.
⚪ 2024-02-08 04:03:11 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-08 04:04:25 UTC Build successful.
⚪ 2024-02-08 04:04:38 UTC Tests are running...
🔴 2024-02-08 04:06:54 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14584 14481 0 12 57 34

@zverevgeny zverevgeny merged commit 112425d into ydb-platform:main Feb 8, 2024
zverevgeny added a commit to zverevgeny/ydb that referenced this pull request Feb 8, 2024
@zverevgeny
YQL-17542 move TaskRunner dependent Execute to TDqSyncComputeActorBase (
5d417ca 
ydb-platform#1666)
@vitstn vitstn mentioned this pull request Feb 8, 2024
Merged
@ildar-khisambeev ildar-khisambeev mentioned this pull request Feb 8, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 8, 2024
Merged
zverevgeny added a commit that referenced this pull request Feb 9, 2024
@zverevgeny
YQL-17542 finalize split sync async CAs (#1689)
ae6ef0c 
* Yql 17542 simplify alloc in compute actor (#1452)

* YQL-17542 Simplify allocator usage in ComputeActors

* fix build

* fix dup

* YQL-17542 move SaveState LoadState (#1474)

* YQL-17703 always use sized allocator in CA (#1522)

* YQL-17542 split FillIoMaps (#1537)

* YQL-17755 fix drying input up (#1604)

* YQL-17542 split stat (#1553)

* YQL-17542 remove transition guards (#1610)

* YQL-17542 get rid of std::any in handling sources state (#1635)

* YQL-17755 ut for TComputeActorAsyncInputHelperTest::PollAsyncInput (#1626)

* YQL-17542 move TaskRunner dependent Execute to TDqSyncComputeActorBase (#1599)

* YQL-17542 move TaskRunner dependent Execute to TDqSyncComputeActorBase (#1666)
@niksaveliev niksaveliev mentioned this pull request Feb 9, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 9, 2024
Merged
@serbel324 serbel324 mentioned this pull request Feb 9, 2024
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 9, 2024
Merged
@niksaveliev niksaveliev mentioned this pull request Feb 12, 2024
Merged
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
Closed
@vitstn vitstn mentioned this pull request Feb 12, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 12, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 12, 2024
Merged
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
Merged
@serbel324 serbel324 mentioned this pull request Feb 13, 2024
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 13, 2024
Merged
@serbel324 serbel324 mentioned this pull request Feb 13, 2024
Merged
This was referenced Feb 13, 2024
Merged
Merged
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 15, 2024
Merged
@niksaveliev niksaveliev mentioned this pull request Feb 16, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 16, 2024
Merged
This was referenced Feb 17, 2024
Merged
Merged
Merged
Merged
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Darych Darych Darych approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

not-for-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zverevgeny @Darych