Skip to content

[k2] add temporary stats for confdata and instance cache calls#1601

Open
apolyakov wants to merge 1 commit intomasterfrom
apolyakov/k2-add-confdata-instance-cache-stats
Open

[k2] add temporary stats for confdata and instance cache calls#1601
apolyakov wants to merge 1 commit intomasterfrom
apolyakov/k2-add-confdata-instance-cache-stats

Conversation

@apolyakov
Copy link
Copy Markdown
Contributor

No description provided.

@apolyakov apolyakov added this to the next milestone Apr 7, 2026
@apolyakov apolyakov requested review from DrDet and PetrShumilov April 7, 2026 16:09
@apolyakov apolyakov self-assigned this Apr 7, 2026
@apolyakov apolyakov added runtime Feature related to runtime k2 Affects compiler or runtime in K2 mode labels Apr 7, 2026
@apolyakov apolyakov force-pushed the apolyakov/k2-add-confdata-instance-cache-stats branch from 6641f35 to dad8e3f Compare April 7, 2026 16:14
Comment on lines +249 to +250
kphp::log::info("instance stats (approximately): confdata -> {}ms, instance cache -> {}ms", confdata_instance_state.time_ns >> 20,
instance_cache_instance_state.time_ns >> 20);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's add ratio to total request time?

@apolyakov apolyakov force-pushed the apolyakov/k2-add-confdata-instance-cache-stats branch from dad8e3f to 737b8f2 Compare April 7, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

k2 Affects compiler or runtime in K2 mode runtime Feature related to runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants