YQ-2918: add processing speed statistics to logs #2720
Conversation
|
⚪
|
|
⚪
|
|
⚪
|
|
⚪
|
| first = false; | ||
| } | ||
| PrintSpeeds(builder, statistics.Stats); |
There was a problem hiding this comment.
Нужно две чиселки печатать. Одна скорость по статистике, а другая скорость на все выполнение query. Полное время выполнения query можно из query.meta достать
There was a problem hiding this comment.
Времени выполнения по статистике нет в v2, сейчас вычисляется из времени начала в job.meta(). Вторая чиселка из CpuTimeUs в статистике: считаем скорость per cpu
| } | ||
| // getting bytes/second = 1'000'000 * bytes/microsecond | ||
| auto speed = (value * 1000000.) / std::max(execTime.MicroSeconds(), ui64{1}); | ||
| builder << ", \"" << statName << postfix << "\": " << speed; |
There was a problem hiding this comment.
размерность бы к скорости дописать bytes/sec
There was a problem hiding this comment.
У нас и так названия полей выглядят как IngressBytesPerSecond / IngressBytesPerCpuPerSecond, плюс тут печаетется жсон, проще жить, когда его не надо дополнительно парсить
|
⚪
|
|
⚪
|
|
⚪ |
Changelog entry
Changelog category
Additional information