Skip to content

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

Merged
elasticsearchmachine merged 4 commits intoelastic:masterfrom
weizijun:add-xcontent-filter-tests
Jan 13, 2022
Merged

Test: add a failed xcontent filter test that will wait for jackson upgrade#81634
elasticsearchmachine merged 4 commits intoelastic:masterfrom
weizijun:add-xcontent-filter-tests

Conversation

@weizijun
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

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

@nik9000
Copy link
Copy Markdown
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
Copy Markdown
Member

nik9000 commented Jan 5, 2022

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

@weizijun
Copy link
Copy Markdown
Contributor Author

weizijun commented Jan 5, 2022

@elasticmachine update branch

@weizijun
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@weizijun
Copy link
Copy Markdown
Contributor Author

@nik9000 can you help to merge this PR?

@romseygeek
Copy link
Copy Markdown
Contributor

@elasticmachine test this please

@romseygeek
Copy link
Copy Markdown
Contributor

@elasticmachine update branch

@romseygeek
Copy link
Copy Markdown
Contributor

@elasticmachine ok to test

@elasticsearchmachine elasticsearchmachine merged commit d07d27e into elastic:master Jan 13, 2022
@weizijun
Copy link
Copy Markdown
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