opendal icon indicating copy to clipboard operation
opendal copied to clipboard

refactor(layers/logging): Don't trigger logigng in heavy IO path

Open Xuanwo opened this issue 6 months ago • 1 comments

Which issue does this PR close?

None

Rationale for this change

Users reported that logging layer can result in heavy CPU usage in profile:

image

What changes are included in this PR?

It's not a good idea to trigger log even while it's a cheap operations (like length.to_string())

Are there any user-facing changes?

Logs are smaller.

Xuanwo avatar Jun 29 '25 05:06 Xuanwo

After this PR:

image

Xuanwo avatar Jun 29 '25 13:06 Xuanwo