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

Revert "ESQL: Expose "_ignored" metadata field" #108864

Merged

Conversation

ivancea
Copy link
Contributor

@ivancea ivancea commented May 21, 2024

Reverting after tests failure: https://gradle-enterprise.elastic.co/s/dpi2eib2x2fj2

Reverts #108770

@ivancea ivancea added >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v8.15.0 labels May 21, 2024
@ivancea ivancea self-assigned this May 21, 2024
@ivancea ivancea requested a review from a team as a code owner May 21, 2024 14:14
Copy link

Documentation preview:

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Collaborator

Hi @ivancea, I've created a changelog YAML for you.

Copy link
Member

@not-napoleon not-napoleon left a comment

Choose a reason for hiding this comment

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

LGTM

@ivancea ivancea added the auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label May 21, 2024
@elasticsearchmachine elasticsearchmachine merged commit 04f1fce into main May 21, 2024
16 checks passed
@elasticsearchmachine elasticsearchmachine deleted the revert-108770-feat/esql-ignored-metadata-field branch May 21, 2024 15:22
ivancea added a commit to ivancea/elasticsearch that referenced this pull request May 21, 2024
elasticsearchmachine pushed a commit that referenced this pull request May 22, 2024
Expose "_ignored" metadata field in ESQL queries.

This is the same code merged here:
#108770 Which got reverted
here: #108864

It was reverted because of a test failure:
https://gradle-enterprise.elastic.co/s/dpi2eib2x2fj2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants