dutyu commented on PR #14750: URL: https://github.com/apache/doris/pull/14750#issuecomment-1334954015
> why the cputimeMS is not added in the method? > > `statistics->add_cpu_ms(_scan_cpu_timer->value() / NANOS_PER_MILLIS);` It will not work if just add this line at method `collect_query_statistics`, maybe we also need to do some job at `scanner_scheduler.cpp` to update `_scan_cpu_timer`, but i'm not familiar with c++, so i just fix these two fields first. You can submit a new pr to fix this problem. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
