Skip to content

Fix SqlSearchIT testAllTypesWithRequestToOldNodes (#88866)#88883

Merged
elasticsearchmachine merged 2 commits intoelastic:mainfrom
Luegg:fix-88866
Jul 28, 2022
Merged

Fix SqlSearchIT testAllTypesWithRequestToOldNodes (#88866)#88883
elasticsearchmachine merged 2 commits intoelastic:mainfrom
Luegg:fix-88866

Conversation

@Luegg
Copy link
Copy Markdown
Contributor

@Luegg Luegg commented Jul 28, 2022

Resolves #88866

@Luegg Luegg added >bug :Analytics/SQL SQL querying labels Jul 28, 2022
@Luegg Luegg requested a review from luigidellaquila July 28, 2022 08:02
@elasticsearchmachine elasticsearchmachine added the Team:QL (Deprecated) Meta label for query languages team label Jul 28, 2022
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-ql (Team:QL)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @Luegg, I've created a changelog YAML for you.

Copy link
Copy Markdown
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

(wondering how it did not fail before...)

@Luegg Luegg added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jul 28, 2022
@Luegg
Copy link
Copy Markdown
Contributor Author

Luegg commented Jul 28, 2022

The branch was never executed because bwcVersion was never on or after 8.4. Only with cutting of the 8.4 branch, 8.4.0 was added to the bwc version to run againt.

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💚 Backport successful

Status Branch Result
8.4

Luegg added a commit to Luegg/elasticsearch that referenced this pull request Jul 28, 2022
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Jul 29, 2022
* upstream/main:
  Add 8.5 migration docs (elastic#88923)
  Script: Reindex & UpdateByQuery Metadata (elastic#88665)
  Remove unused plugins dir var from server CLI (elastic#88917)
  Use tracing API in TaskManager (elastic#88885)
  Add source fallback for keyword fields using operation (elastic#88735)
  Prune changelogs after 8.3.3 release
  Bump versions after 8.3.3 release
  Add a test for checking for misspelled "dry_run" parameters for Desired Nodes API (elastic#88898)
  Speedup BalanceUnbalancedClusterTests (elastic#88794)
  Preventing exceptions on node shutdown in integration tests (elastic#88827)
  Do not trigger check part3 for test mute and docs PRs (elastic#88895)
  Add troubleshooting docs about data corruption (elastic#88760)
  Mute RollupActionSingleNodeTests#testRollupDatastream (elastic#88891)
  [DOCS] Domain splitting impacts API keys (elastic#88677)
  Fix SqlSearchIT testAllTypesWithRequestToOldNodes (elastic#88866) (elastic#88883)
  Update synthetic-source.asciidoc (elastic#88880)
  Log more details in TaskAssertions (elastic#88864)
  Make Tuple a record (elastic#88280)
@mark-vieira mark-vieira added v8.4.0 and removed v8.4.1 labels Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/SQL SQL querying auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >bug Team:QL (Deprecated) Meta label for query languages team v8.4.0 v8.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SqlSearchIT testAllTypesWithRequestToOldNodes failing

4 participants