Skip to content
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

Yql 17542 simplify alloc in compute actor #1452

Merged
merged 3 commits into from
Feb 1, 2024
Merged

Yql 17542 simplify alloc in compute actor #1452

merged 3 commits into from
Feb 1, 2024

Conversation

zverevgeny
Copy link
Collaborator

@zverevgeny zverevgeny requested a review from a team as a code owner January 31, 2024 09:35
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 09:38:11 UTC Pre-commit check for 3d49176 has started.
⚪ 2024-01-31 09:38:14 UTC Build linux-x86_64-relwithdebinfo is running...
🔴 2024-01-31 09:40:29 UTC Build failed. see the build logs.
⚪ 2024-01-31 09:40:44 UTC Tests are running...
🔴 2024-01-31 09:51:47 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
37666 29834 0 272 7529 31

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 09:39:29 UTC Pre-commit check for 3d49176 has started.
⚪ 2024-01-31 09:39:31 UTC Build linux-x86_64-release-asan is running...
🔴 2024-01-31 09:41:30 UTC Build failed. see the build logs.
⚪ 2024-01-31 09:41:41 UTC Tests are running...
🔴 2024-01-31 09:49:26 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
7476 7196 0 267 2 11

@zverevgeny zverevgeny force-pushed the YQL-17542_simplify_alloc_in_compute_actor branch from 891a5cb to 7a32c18 Compare January 31, 2024 11:00
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 11:02:18 UTC Pre-commit check for 67c591b has started.
⚪ 2024-01-31 11:02:20 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-31 11:04:12 UTC Build successful.
⚪ 2024-01-31 11:04:25 UTC Tests are running...
🔴 2024-01-31 12:37:14 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16022 15926 0 17 50 29

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 11:04:04 UTC Pre-commit check for 67c591b has started.
⚪ 2024-01-31 11:04:07 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-31 11:06:19 UTC Build successful.
⚪ 2024-01-31 11:06:34 UTC Tests are running...
🔴 2024-01-31 12:30:06 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60345 51038 0 2 9270 35

@zverevgeny zverevgeny requested review from resetius and Darych January 31, 2024 15:51
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 17:43:43 UTC Pre-commit check for 27af424 has started.
⚪ 2024-01-31 17:43:44 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-31 17:47:37 UTC Build successful.
⚪ 2024-01-31 17:47:46 UTC Tests are running...
🔴 2024-01-31 19:14:54 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60356 51056 0 1 9270 29

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 31, 2024 •
edited
Loading

⚪ 2024-01-31 17:51:58 UTC Pre-commit check for 27af424 has started.
⚪ 2024-01-31 17:51:59 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-31 17:54:25 UTC Build successful.
⚪ 2024-01-31 17:54:37 UTC Tests are running...
🔴 2024-01-31 19:30:22 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
16046 15916 0 23 67 40

@zverevgeny zverevgeny merged commit 9c7d88b into ydb-platform:main Feb 1, 2024
2 of 4 checks passed
This was referenced Feb 1, 2024
Merged
Merged
Merged
Merged
Merged
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 3, 2024
Merged
@FloatingCrowbar FloatingCrowbar mentioned this pull request Feb 4, 2024
Merged
@aakulaga-ydb aakulaga-ydb mentioned this pull request Feb 4, 2024
Merged
@shnikd shnikd mentioned this pull request Feb 5, 2024
Closed
@niksaveliev niksaveliev mentioned this pull request Feb 5, 2024
Merged
@shnikd shnikd mentioned this pull request Feb 5, 2024
Merged
This was referenced Feb 5, 2024
Merged
Merged
zverevgeny added a commit to zverevgeny/ydb that referenced this pull request Feb 8, 2024
@zverevgeny
Yql 17542 simplify alloc in compute actor (ydb-platform#1452)
382f31d 
* YQL-17542 Simplify allocator usage in ComputeActors

* fix build

* fix dup
This was referenced Feb 8, 2024
Merged
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
@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
@pavelvelikhov pavelvelikhov 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
@aakulaga-ydb aakulaga-ydb mentioned this pull request Feb 24, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 26, 2024
Merged
lll-phill-lll added a commit to lll-phill-lll/ydb that referenced this pull request Apr 25, 2024
@lll-phill-lll
Revert "Yql 17542 simplify alloc in compute actor (ydb-platform#1452)"
01a0c19 
This reverts commit 9c7d88b.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Darych Darych Darych approved these changes

@resetius resetius Awaiting requested review from resetius

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
@zverevgeny @Darych