daradurvs commented on a change in pull request #8431:
URL: https://github.com/apache/ignite/pull/8431#discussion_r531131279
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceInfo.java
##########
@@ -37,6 +39,9 @@
/** */
private static final long serialVersionUID = 0L;
+ /** Context. */
+ private final GridKernalContext ctx;
Review comment:
I've tried to cover the case with statically configured services, which
will be received during node joining.
In this case server (not user) will deserialize `ServiceInfo` without
calling the new constructor.
Look at:
https://github.com/apache/ignite/blob/8f824ca5822d4d7d0b9eb12aa726c32a56776bbc/modules/core/src/main/java/org/apache/ignite/internal/processors/service/IgniteServiceProcessor.java#L350
I think we need a test for this case too.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
