Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[!!!][TASK] Split SolrService into SolrAdminService, SolrReadService and SolrWriteService #1746

Conversation

timohund
Copy link
Contributor

@timohund timohund commented Nov 21, 2017

This pr:

  • Extracts the SolrAdminService, SolrReadService and SolrWriteService from the SolrService.
  • Changes the SolrConnectionManager to return a SolrConnection that could be used to access these services.
  • Allows as followup to differnciate between read and write connections an to send them to different solr servers (master/slave)

Impact:

The ConnectionManager now return a SolrConnection that has the methods "getReadService", "getWriteService" and get "getAdminService".
These Services contain the corresponding method from the previous SolrService.

Fixes: #1745

@timohund timohund added this to the 8.0 - UX Improvements milestone Nov 21, 2017
@timohund timohund force-pushed the task/master/1745-split-SolrService-into-read-write-admin branch 4 times, most recently from f8c5a3a to d553ad7 Compare November 24, 2017 11:20
@timohund timohund changed the title [WIP][TASK] Split SolrService into SolrAdminService, SolrReadService and SolrWriteService [!!!][TASK] Split SolrService into SolrAdminService, SolrReadService and SolrWriteService Nov 24, 2017
@timohund timohund force-pushed the task/master/1745-split-SolrService-into-read-write-admin branch from d553ad7 to 2ef302c Compare November 24, 2017 11:53
…and SolrWriteService

This pr:

* Extracts the SolrAdminService, SolrReadService and SolrWriteService from the SolrService.
* Changes the SolrConnectionManager to return a SolrConnection that could be used to access these services.
* Allows as followup to differnciate between read and write connections an to send them to different solr servers (master/slave)

Impact:

The ConnectionManager now return a SolrConnection that has the methods "getReadService", "getWriteService" and get "getAdminService".
These Services contain the corresponding method from the previous SolrService.

Fixes: TYPO3-Solr#1745
@timohund timohund force-pushed the task/master/1745-split-SolrService-into-read-write-admin branch from 2ef302c to eef6386 Compare November 24, 2017 15:22
Copy link
Collaborator

@dkd-kaehm dkd-kaehm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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.

None yet

2 participants