Skip to content

Improve performance of log message prettification function #230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
coderabbitai bot opened this issue Apr 10, 2025 · 0 comments · May be fixed by #225
Open

Improve performance of log message prettification function #230

coderabbitai bot opened this issue Apr 10, 2025 · 0 comments · May be fixed by #225

Comments

@coderabbitai
Copy link

coderabbitai bot commented Apr 10, 2025

Description

The current implementation of log message prettification uses the 'js-beautify' library, which works well for the initial implementation but may have performance concerns when working with large log files.

Background

This feature was initially implemented in PR #223, which added toggle functionality for prettifying log messages. PR #225 has made some related changes to this functionality.

Objective

Investigate and implement performance improvements for the log message prettification feature:

  1. Evaluate the performance of the current js-beautify implementation with different log file sizes
  2. Consider alternatives or optimizations that could improve performance
  3. Implement custom solution that prioritizes performance with large log files
  4. Add appropriate tests to verify performance characteristics

Related

@junhaoliao

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 a pull request may close this issue.

1 participant