search

matrixorigin / matrixone

Hyperconverged cloud-edge native database
https://docs.matrixorigin.cn/en
Apache License 2.0
1.79k stars 277 forks source link

fileservice: refactor http trace #20177

Closed reusee closed 1 day ago

reusee commented 2 days ago

What type of PR is this?

Which issue(s) this PR fixes:

issue #20129 #20221

What this PR does / why we need it:

refactor HTTP trace to reduce cost.

reusee commented 1 day ago

What exactly is the change? Do we have a number of how much/big is the improvement?

现在的代码，即使 Read 命中缓存，没有任何http操作，也一样会初始化httptrace。这个pr改成将httptrace放在object storage层，只在实际发生http操作时，才加上。开销是必然会降低的。

另外现在的代码未覆盖所有可能涉及http操作的路径，这个pr修正了。