Skip to content

Allow parameter for ES|QL SAMPLE #129392

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

Merged
merged 3 commits into from
Jun 17, 2025

Conversation

jan-elastic
Copy link
Contributor

No description provided.

@jan-elastic jan-elastic added >non-issue :ml Machine learning Team:ML Meta label for the ML team v9.1.0 labels Jun 13, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@jan-elastic jan-elastic force-pushed the esql-sample-parameter branch 3 times, most recently from 4881a8b to c38e9f3 Compare June 13, 2025 10:49
@jan-elastic jan-elastic force-pushed the esql-sample-parameter branch from 48c40a6 to 7236044 Compare June 17, 2025 08:30
Copy link
Contributor

@luigidellaquila luigidellaquila left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jan-elastic jan-elastic merged commit 3f9bf9a into elastic:main Jun 17, 2025
27 checks passed
jan-elastic added a commit to jan-elastic/elasticsearch that referenced this pull request Jun 18, 2025
* Allow parameter for ES|QL SAMPLE

* fix test to work around issue elastic#120272

* remove unused postAnalysisVerification
jan-elastic added a commit to jan-elastic/elasticsearch that referenced this pull request Jun 18, 2025
* Allow parameter for ES|QL SAMPLE

* fix test to work around issue elastic#120272

* remove unused postAnalysisVerification
jan-elastic added a commit to jan-elastic/elasticsearch that referenced this pull request Jun 18, 2025
* Allow parameter for ES|QL SAMPLE

* fix test to work around issue elastic#120272

* remove unused postAnalysisVerification
jan-elastic added a commit to jan-elastic/elasticsearch that referenced this pull request Jun 18, 2025
* Allow parameter for ES|QL SAMPLE

* fix test to work around issue elastic#120272

* remove unused postAnalysisVerification
jan-elastic added a commit to jan-elastic/elasticsearch that referenced this pull request Jun 19, 2025
* Allow parameter for ES|QL SAMPLE

* fix test to work around issue elastic#120272

* remove unused postAnalysisVerification
jan-elastic added a commit that referenced this pull request Jun 19, 2025
* ES|QL random sampling  (#125570)

* test: check ES|QL SAMPLE capability before running analyzer/parser tests (#127382)

This commit resolves release test failures by checking the ES|QL SAMPLE capability before running the tests.

* Fix ES|QL sample csv tests (#127838)

* Remove optional seed from ES|QL SAMPLE (#128887)

* Remove optional seed from ES|QL SAMPLE

* make it clear that seed is for testing

* rename ES|QL sample capability (#129193)

* remove stats correction from ES|QL sample (#129319)

* remove stats correction from ES|QL sample

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>

* Allow parameter for ES|QL SAMPLE (#129392)

* Allow parameter for ES|QL SAMPLE

* fix test to work around issue #120272

* remove unused postAnalysisVerification

* docs

* [CI] Auto commit changes from spotless

* fix usage

* Move ES|QL sample command from snapshot to tech preview (#129540)

* Move ES|QL sample command from snapshot to tech preview

* dev_sample -> sample

* fix test

* fix grammar

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning >non-issue Team:ML Meta label for the ML team v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants