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

ESQL: Properly skip datasets with lookup indices #118682

Open
wants to merge 1 commit into
base: 8.x
Choose a base branch
from

Conversation

alex-spies
Copy link
Contributor

Do not rely on specific naming, read the actual settings.

This is required to unblock the backport #118661, which adds a new lookup index dataset whose name doesn't end with _lookup.

When this is merged, I plan to manually forward port it to main; the PR is against 8.x because this is where the problems occur.

Do not rely on specific naming, read the actual settings.
@alex-spies alex-spies added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v8.18.0 labels Dec 13, 2024
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 13, 2024
Copy link
Contributor

@craigtaverner craigtaverner left a comment

Choose a reason for hiding this comment

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

LGTM. Not sure why the stream was removed, but I guess it looks more classic-java now!

}

public static boolean isLookupDataset(TestsDataset dataset) throws IOException {
Settings settings = dataset.readSettingsFile();
Copy link
Contributor

Choose a reason for hiding this comment

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

OK, that was easier than expected!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants