Skip to content

Conversation

@khushijain21
Copy link
Contributor

@khushijain21 khushijain21 commented Jul 28, 2025

What does this PR do?

Development/Testing logger by logp library prints stack strace at error/warn level which can be mistaken for a panic. To avoid this confusion, we disbale the stack traces in this PR.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works

Related issues

@khushijain21 khushijain21 marked this pull request as ready for review July 28, 2025 09:31
@khushijain21 khushijain21 requested a review from a team as a code owner July 28, 2025 09:31
@khushijain21 khushijain21 requested review from andrzej-stencel, belimawr and mauri870 and removed request for a team July 28, 2025 09:31
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@khushijain21 khushijain21 self-assigned this Jul 28, 2025
Copy link
Member

@mauri870 mauri870 left a comment

Choose a reason for hiding this comment

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

LGTM, if I understand this correctly it is simply restoring previous behavior.

@khushijain21 khushijain21 merged commit 2a4dfe3 into elastic:main Jul 28, 2025
5 checks passed
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.

3 participants