Skip to content

painless error log message updated to a more informative one fix for issue #28820 #74533

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

Closed
wants to merge 1 commit into from
Closed

painless error log message updated to a more informative one fix for issue #28820 #74533

wants to merge 1 commit into from

Conversation

larissabinatti
Copy link

@larissabinatti larissabinatti commented Jun 24, 2021

Trying to fix the logging problem from issue #28820

@elasticsearchmachine elasticsearchmachine added the external-contributor Pull request authored by a developer outside the Elasticsearch team label Jun 24, 2021
@mark-vieira mark-vieira added the :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache label Jun 25, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jun 25, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@larissabinatti larissabinatti changed the title painless error log message updated to a more informative one painless error log message updated to a more informative one fix for issue #28820 Jul 1, 2021
Copy link
Contributor

@stu-elastic stu-elastic left a comment

Choose a reason for hiding this comment

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

Hi @larissabinatti can you try the reproduction from #28820 (comment) and include a few tests to ensure your change addresses the issue?

@larissabinatti larissabinatti deleted the painlesserror-log-fix branch July 13, 2021 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Core/Infra Meta label for core/infra team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants