-
Notifications
You must be signed in to change notification settings - Fork 606
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
YQ-3970 RD fixed memory leak in filters #12756
YQ-3970 RD fixed memory leak in filters #12756
Conversation
Fixed purecalc cache invalidation commit_hash:088ac16b3481db39893f4fab0cf80c3809a5dcdd
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
@@ -77,6 +77,7 @@ struct TComputationMutables { | |||
std::vector<ui32> SerializableValues; // Indices of values that need to be saved in IComputationGraph::SaveGraphState() and restored in IComputationGraph::LoadGraphState(). | |||
ui32 CurWideFieldsIndex = 0U; | |||
std::vector<TWideFieldsInitInfo> WideFieldInitialize; | |||
std::vector<ui32> CachedValues; // Indices of values that holds temporary cached data and unreachable by dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Правки в /yql/essentials надо вносить в аркадию сначала
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
и /yt тоже
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сорри, это ваш бранч, делайте что хотите:)
Предлагаю поправить CODEOWNERS чтобы не было ревьюеров обычных
4dd4793
into
ydb-platform:q-stable-2024-12-05
Changelog entry
Fixed memory leak in filters
Changelog category
Additional information