Skip to content
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

Test: add a failed xcontent filter test that will wait for jackson upgrade #81634

Merged

Conversation

weizijun
Copy link
Contributor

add a xcontent filter test that will failed, when random to filterOnParser.

The test will fixed when jackson upgrade in #80160

@weizijun weizijun changed the title add a failed xcontent filter test that will wait for jackson upgrade Test: add a failed xcontent filter test that will wait for jackson upgrade Dec 12, 2021
@elasticsearchmachine elasticsearchmachine added v8.1.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Dec 12, 2021
@jtibshirani jtibshirani added the :StorageEngine/TSDB You know, for Metrics label Dec 13, 2021
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 13, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@nik9000
Copy link
Member

nik9000 commented Jan 5, 2022

@elasticmachine, test this please

@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Jan 5, 2022
@nik9000
Copy link
Member

nik9000 commented Jan 5, 2022

labelbot, please wake up. I've update the labels.

@weizijun
Copy link
Contributor Author

weizijun commented Jan 5, 2022

@elasticmachine update branch

@weizijun
Copy link
Contributor Author

@elasticmachine update branch

@weizijun
Copy link
Contributor Author

@nik9000 can you help to merge this PR?

@romseygeek
Copy link
Contributor

@elasticmachine test this please

@romseygeek
Copy link
Contributor

@elasticmachine update branch

@romseygeek
Copy link
Contributor

@elasticmachine ok to test

@elasticsearchmachine elasticsearchmachine merged commit d07d27e into elastic:master Jan 13, 2022
@weizijun
Copy link
Contributor Author

@romseygeek Thanks for the merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) external-contributor Pull request authored by a developer outside the Elasticsearch team :StorageEngine/TSDB You know, for Metrics Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants