Skip to content

Conversation

@craigtaverner
Copy link
Contributor

Backports the following commits to 8.14:

When trying to use a gradle `skipTest` rule on tests named "values", we get a class-caste exception in Jackson.

This PR needs to rename this function for all versions of Elasticsearch that the `yamlRestCompatTestTransform` task will run on, so that later PRs that add skipTests will be able to pass. Since this test was added in 8.14, we must backport all the way back to there.

```
class com.fasterxml.jackson.databind.node.IntNode cannot be cast to class com.fasterxml.jackson.databind.node.ArrayNode (com.fasterxml.jackson.databind.node.IntNode and com.fasterxml.jackson.databind.node.ArrayNode are in unnamed module of loader org.gradle.internal.classloader.VisitableURLClassLoader$InstrumentingVisitableURLClassLoader @50337c96)
```
@craigtaverner craigtaverner added :Analytics/ES|QL AKA ESQL >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Oct 18, 2024
@craigtaverner
Copy link
Contributor Author

@elasticmachine update branch

@elasticmachine
Copy link
Collaborator

There are no new commits on the base branch.

@craigtaverner craigtaverner merged commit 09c8a68 into elastic:8.14 Oct 18, 2024
12 of 16 checks passed
@craigtaverner craigtaverner deleted the backport/8.14/pr-115096 branch October 18, 2024 16:21
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-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.14.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants