Skip to content

Add integration tests that verify ST_EXTENT_AGG needs a license #125584

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
Mar 26, 2025

Conversation

craigtaverner
Copy link
Contributor

Modelled on the similar work for verifying RRF license checks.

@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Mar 25, 2025
@craigtaverner craigtaverner requested a review from ioanatia March 25, 2025 13:55
@elasticsearchmachine
Copy link
Collaborator

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

@craigtaverner craigtaverner requested a review from iverase March 26, 2025 17:10
@craigtaverner craigtaverner merged commit 220f4c4 into elastic:main Mar 26, 2025
17 checks passed
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Mar 26, 2025
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Mar 26, 2025
elasticsearchmachine pushed a commit that referenced this pull request Mar 27, 2025
The random license selection in
#125584 was incorrectly
assuming ENTERPRISE was needed, when it should have been PLATINUM. This
resulted in occasional test failures (expected to fail on PLATINUM, and
succeeded).

Fixes #125734
elasticsearchmachine pushed a commit that referenced this pull request Mar 28, 2025
…#125584) (#125712) (#125719)

* Add integration tests that verify ST_EXTENT_AGG needs a license (#125584) (#125712)

* Fix java version problem on backport

* Use PLATINUM as license level for spatial features
craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Mar 28, 2025
…elastic#125584) (elastic#125712) (elastic#125719)

* Add integration tests that verify ST_EXTENT_AGG needs a license (elastic#125584) (elastic#125712)

* Fix java version problem on backport

* Use PLATINUM as license level for spatial features
elasticsearchmachine pushed a commit that referenced this pull request Mar 28, 2025
…#125584) (#125712) (#125719) (#125834)

* Add integration tests that verify ST_EXTENT_AGG needs a license (#125584) (#125712)

* Fix java version problem on backport

* Use PLATINUM as license level for spatial features
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
The random license selection in
elastic#125584 was incorrectly
assuming ENTERPRISE was needed, when it should have been PLATINUM. This
resulted in occasional test failures (expected to fail on PLATINUM, and
succeeded).

Fixes elastic#125734
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 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants