zstan commented on code in PR #12666:
URL: https://github.com/apache/ignite/pull/12666#discussion_r2753277136


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/store/GridCacheStoreManagerAdapter.java:
##########
@@ -117,12 +120,17 @@ public abstract class GridCacheStoreManagerAdapter 
extends GridCacheManagerAdapt
     /** Always keep binary. */
     protected boolean alwaysKeepBinary;
 
+    /** Failure handler reaction. */
+    private Consumer<Throwable> failureHandlerAction;
+
     /** {@inheritDoc} */
     @SuppressWarnings("unchecked")
     @Override public void initialize(@Nullable CacheStore cfgStore, Map 
sesHolders) throws IgniteCheckedException {
         GridKernalContext ctx = igniteContext();
         CacheConfiguration cfg = cacheConfiguration();
 
+        failureHandlerAction = e -> ctx.failure().process(new 
FailureContext(FailureType.CRITICAL_ERROR, e));

Review Comment:
   removed



##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/store/GridCacheStoreManagerAdapter.java:
##########
@@ -914,6 +922,13 @@ private void notifyCacheStoreSessionListeners(SessionData 
ses, @Nullable StoreOp
                     lsnr.onSessionStart(locSes);
             }
         }
+        catch (RuntimeException e) {
+            U.error(log, "Exception raised during notify SessionListeners: ", 
e);

Review Comment:
   changed



-- 
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]

Reply via email to