Remove top-level size from ReindexRequest#5471
Conversation
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373).
|
Following you can find the validation changes against the target branch for the APIs.
You can validate these APIs yourself by using the |
|
Thanks! Is it silently accepted, or does the server refuse it? I'd like to feel less bad about the backports :) |
The server has thrown an error since 8.0.0, based on my reading of the PRs. |
|
Hi @pquentin, can you advise on the backporting here? I think this falls under
because the API is absolutely usable, the only issue is that clients may provide an option that would result in a 400. But I may be interpreting that wrongly. |
|
Since this option is unusable, backporting is fine! |
|
Okay, backporting is also fine with me. Are you okay to approve this PR, or should I ask someone from my team? |
|
The backport to To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-8.19 8.19
# Navigate to the new working tree
cd .worktrees/backport-8.19
# Create a new branch
git switch --create backport-5471-to-8.19
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 263280a58bdaa8ee1061eb407bf2280bd6547207
# Push it to GitHub
git push --set-upstream origin backport-5471-to-8.19
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-8.19Then, create a pull request where the |
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373). (cherry picked from commit 263280a)
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373). (cherry picked from commit 263280a)
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373). (cherry picked from commit 263280a)
|
Do backports in this repo need review? |
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373). (cherry picked from commit 263280a) Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
This was removed from ES in 8.0.0 (see elastic/elasticsearch#43373). (cherry picked from commit 263280a) Co-authored-by: Pete Gillin <pete.gillin@elastic.co>
This was removed from ES in 8.0.0 (see
elastic/elasticsearch#43373).