Skip to content

Conversation

Watson1978
Copy link
Contributor

@Watson1978 Watson1978 commented Apr 1, 2025

Which issue(s) this PR fixes:
Fixes #

What this PR does / why we need it:
When dealing with long logs at fast, it might improve memory efficient to clear the original strings used in string interpolation.

This is similar with #4884 .

chart

  • config
<source>
  @type sample
  tag test
  sample {"message": "#{'a' * 100 * 1024}"}
  size 100
</source>

<match **>
  @type file
  path "#{File.expand_path '~/tmp/log'}"
  format json
</match>

Docs Changes:

Release Note:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978 Watson1978 marked this pull request as ready for review April 1, 2025 00:52
@Watson1978 Watson1978 requested a review from daipom April 1, 2025 00:52
Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

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

Thanks!

@daipom daipom added this to the v1.19.0 milestone Apr 1, 2025
@daipom daipom merged commit 7704d75 into fluent:master Apr 1, 2025
9 of 10 checks passed
@Watson1978 Watson1978 deleted the formatter_json branch April 1, 2025 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants