[TASK:BP:11.6] Relax dependencies to local tables #3765
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As there may be several indexing configurations per type the queue check has to consider the indexing configuration. The missing check is added by this commit. Ports: TYPO3-Solr#3764 Resolves: TYPO3-Solr#3763
As an initial steps to allow custom index queue types interfaces for queue and queue items are introduced. Additionally custom queue classes can be configured via: plugin.tx_solr.index.queue.<configurationName>.indexQueue This is a breaking change for all instances that implemented own queues and queue items! Ports: TYPO3-Solr#3764 Resolves: TYPO3-Solr#3763
Queue initialization status returned by the QueueInitializationService isn't considered in the backend module and a success message is always shown. This commit adds a check for the returned status and displays an error message similiar to the message that is shown if an exceptions occured. Ports: TYPO3-Solr#3764 Resolves: TYPO3-Solr#3763
Existing exceptions are now based on the recently introduced generic EXT:solr exception. Additionally based on the new generic EXT:solr exception a further exception is introduced, allowing more targeted handling of exceptions: InvalidIndexQueueInitizalizationPostProcessorException This is a breaking change as the base excepion changed and the new exception is no longer based on \UnexpectedValueException. Ports: TYPO3-Solr#3764 Resolves: TYPO3-Solr#3763
|
DO NOT MERGE! Base has to be 11.6! |
10 tasks
dkd-kaehm
approved these changes
Sep 15, 2023
dkd-kaehm
deleted the
task/release-11.6.x/3763-relax_dependency_to_local_tables
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #3764
DO NOT MERGE! Base has to be 11.6!
What this pr does
To allow indexing of external data the dependencies to local tables are relaxed, 4 steps are prepared to achieve this:
Most of these changes are breaking, as internal interfaces and exceptions are changed, that might be used outside of EXT:solr.
How to test
This changes just prepares EXT:solr for indexing of external data, the current behaviour is not changed!
Resolves: #3763