Skip to content

distrib join through coord node#4184

Draft
mkhludnev wants to merge 4 commits intoapache:mainfrom
mkhludnev:distribJoinCoordNode
Draft

distrib join through coord node#4184
mkhludnev wants to merge 4 commits intoapache:mainfrom
mkhludnev:distribJoinCoordNode

Conversation

@mkhludnev
Copy link
Member

https://issues.apache.org/jira/browse/SOLR-XXXXX

Description

Please provide a short description of the changes you're making with this pull request.

Solution

Please provide a short description of the approach taken to implement your solution.
We appreciate transparent disclosure of significant AI Coding assistants usage.

Tests

Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide
  • I have added a changelog entry for my change

@mkhludnev
Copy link
Member Author

@endika-ravn
Copy link

#4186 There's a different behaviour (or at least seems to be) between coordinator nodes and data nodes that act as proxy. I have modified the tests so the coordinator node is explicitly started as a coordinator.

@dsmiley
Copy link
Contributor

dsmiley commented Mar 7, 2026

CC @patsonluk

@mkhludnev
Copy link
Member Author

It seem like the code here #4184 forwards request to the data node. At least, it's worth to clarify "coordinator" expectation/behavior in the guide. Real trouble with coordinator-only join occurs at #4186

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants