Revert "Return error when remote indices are locally resolved (#74762)"#76798
Revert "Return error when remote indices are locally resolved (#74762)"#76798jtibshirani merged 1 commit intoelastic:7.xfrom
Conversation
|
Pinging @elastic/es-search (Team:Search) |
dnhatn
left a comment
There was a problem hiding this comment.
LGTM. Can we also mark the original PR as non-issue so it won't make into the release notes?
|
Will do! Thanks for reviewing. |
|
Thanks for taking care of this @jtibshirani . I did some digging and I understand better what happened. In our main branch any remote cluster expression will be intercepted and converted directly when resolving remote cluster names, hence such expressions won't ever make it to the index name expression resolver which resolves local indices names. In our 7.x branches though, we still account for indices created in 6.x that may contain a |
This commit is causing searches that use remote patterns like
*:testto failwith the error "Cross-cluster calls are not supported in this context but
remote indices were requested: [*:test]". I don't think this was the intention
of the original PR, which targeted APIs like 'get' that do not support
cross-cluster calls.
This is blocking release testing, so I'm reverting for now. We can re-introduce
the PR later along with a test for this search case.