Skip to content

Added support for multiple cores.#1

Open
adrianschulz wants to merge 1 commit into
pointfield-supportfrom
multicore-support
Open

Added support for multiple cores.#1
adrianschulz wants to merge 1 commit into
pointfield-supportfrom
multicore-support

Conversation

@adrianschulz

Copy link
Copy Markdown
Owner

No description provided.

@tisto

tisto commented May 19, 2020

Copy link
Copy Markdown

@adrianschulz could you elaborate your use case a bit more? Solr allows you to search across multiple cores, so you could move the abstraction to Solr level instead of Plone if I am not mistaken.

@adrianschulz

Copy link
Copy Markdown
Owner Author

@tisto Sure!

https://stackoverflow.com/questions/39450164/how-to-search-multiple-solr-core
Like you said it is possible to query across multiple cores with solr. But you have some limitations (strange scoring, evenly distribution necessary). Also you have exactly 1 result-set (results from both cores merged).
What we needed is a result-set per core (not merged!) because we want to display them as different result-sets. For this reason we need to be able to specify a core for each query. In my opinion this is clearly a thing for collective.solr. So I added arguments in some places which allows exactly what we wanted.

@tisto

tisto commented May 21, 2020

Copy link
Copy Markdown

@adrianschulz thanks for the clarification. This makes sense. I just want to understand the use case, that's all. :)

Would it be possible to write a unit/integration test for this? Please also add a changelog entry.

@adrianschulz

Copy link
Copy Markdown
Owner Author

@tisto

Sure. But since I'm quite busy at the moment I will need some weeks for it (same for the other pull request).

@adrianschulz

Copy link
Copy Markdown
Owner Author

@tisto

I noticed that the development of this without breaking old collective.solr-setups is more difficult than I thought. So I think I need a lot longer than I planned.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants