alex-plekhanov commented on code in PR #12453:
URL: https://github.com/apache/ignite/pull/12453#discussion_r2483157718
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/query/GridQueryProcessor.java:
##########
@@ -2350,7 +2365,7 @@ private void registerCache0(
boolean isSql
) throws IgniteCheckedException {
synchronized (stateMux) {
- if (moduleEnabled()) {
+ if (moduleEnabled() && !cacheNamesToSchema.containsKey(cacheName))
{
Review Comment:
It's not optimization, it's change required for correct work. This check was
moved from `prepareChangeOnNotStartedCache`, because we always need to register
query entity (query entity registration now allowed for already registered
cache), but cache registration required only once.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]