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

KQL update tests #116483

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Conversation

afoucret
Copy link
Contributor

@afoucret afoucret commented Nov 8, 2024

This PR updates the KQL tests to prepare the implementation of nested field support.

More specifically it does:

  • use field name that are mapped in the tests instead of random field names in supported-queries and unsupported-queries
  • Add nested field and subfields to the mapping used in tests

@afoucret afoucret added >non-issue auto-backport Automatically create backport pull requests when merged Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch :Search Relevance/Search Catch all for Search Relevance v9.0.0 v8.17.0 labels Nov 8, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

foo_*_field:foo
foo_field:*
foo_*:*
mapped_int:200
Copy link
Contributor

Choose a reason for hiding this comment

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

I would have named these int_field, text_field etc - but these were already added in AbstractBuilderTestCase a long time ago, so it's all good.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would also prefer it but it is inherited, so I have to deal with these names 😿

mapping.field("type", "nested");
mapping.startObject("properties");
{
mapping.startObject(TEXT_FIELD_NAME).field("type", "text").endObject();
Copy link
Contributor

Choose a reason for hiding this comment

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

quite the nesting 😄

@afoucret afoucret merged commit 6eb7729 into elastic:main Nov 8, 2024
16 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x

afoucret added a commit to afoucret/elasticsearch that referenced this pull request Nov 8, 2024
elasticsearchmachine pushed a commit that referenced this pull request Nov 8, 2024
jozala pushed a commit that referenced this pull request Nov 13, 2024
alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >non-issue :Search Relevance/Search Catch all for Search Relevance Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants