code
code
I still think it's weird to include .cc in a .h file (Although I know there are some similar cases in real world. But obviously, we can avoid that here)....
Force update is used because re-organization of commits.
The benchmark result of previous stats handler: ``` ---------------------------------------------------------------------------------------------- Benchmark Time CPU Iterations ---------------------------------------------------------------------------------------------- BM_AllCountersJson/per_endpoint_stats_disabled 590 ms 590 ms 1 output per iteration: 135789011 [2024-08-28 10:20:15.902][265134][error] [test/server/admin/stats_handler_speed_test.cc:163] Initializing cluster info;...
Replaced by #36001
could you check the CI? Thanks.
Fine, most comments are addressed. There are two problems: - the substitution_formatter.h is depended by the envoy-mobile. But the `sanitize` introduce a exception on the core data path. - `serializeDubbo`...