shishkovilja commented on code in PR #13095:
URL: https://github.com/apache/ignite/pull/13095#discussion_r3638049172
##########
modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java:
##########
@@ -1318,15 +1319,19 @@ else if (e instanceof IgniteCheckedException)
startTimer.finishGlobalStage("Await exchange");
}
+ /** Test entry to {@link #initMessageFactory()}, which production reaches
via {@link #start}. */
+ @TestOnly
+ public void initMessageFactoryForTest() throws IgniteCheckedException {
Review Comment:
Why we adding such method? It is used only in one test. Let's remove it.
`initMessageFactory` can be invoked in the test via `U.invoke`:
`U.invoke(IgniteKernal.class, cacheIoMgr.context().kernalContext().grid(),
"initMessageFactory");`
--
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]
