-
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
YQL-18314 use shared alloc in all task runners #4233
YQL-18314 use shared alloc in all task runners #4233
Conversation
@@ -1299,8 +1300,7 @@ class TTaskRunner: public IPipeTaskRunner { | |||
: TraceId(traceId) | |||
, Task(task) | |||
, FilesHolder(std::move(filesHolder)) | |||
, Alloc(new NKikimr::NMiniKQL::TScopedAlloc(__LOCATION__, NKikimr::TAlignedPagePoolCounters(), true), |
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.
this is the problem being fixed
⚪ |
⚪ |
⚪ |
⚪ |
⚪ |
⚪ |
⚪
|
⚪
|
|
||
const TMap<ui64, const NYql::NDq::TDqTaskRunnerStats*> GetTasksStats() const { return Stats; } | ||
private: | ||
TMap<ui64, TIntrusivePtr<NYql::NDq::IDqTaskRunner>> TaskRunners; | ||
TMap<ui64, NYql::NDq::TDqTaskSettings> Tasks; | ||
TMap<ui64, const NYql::NDq::TDqTaskRunnerStats*> Stats; | ||
NYql::NDq::TLogFunc LogFunc; | ||
NMiniKQL::TScopedAlloc& Alloc; | ||
std::shared_ptr<NKikimr::NMiniKQL::TScopedAlloc> Alloc; |
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.
indent кажется поплыл
0114bce
to
b6be2cb
Compare
⚪
|
⚪
|
⚪
|
⚪
|
e21e733
to
4d34664
Compare
⚪
|
⚪
|
No description provided.