[
https://issues.apache.org/jira/browse/IGNITE-3548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15962980#comment-15962980
]
Pavel Tupitsyn commented on IGNITE-3548:
----------------------------------------
[~vozerov] {{LifecycleBean}} is not acceptable either. It just does not make
sense for a non-Java people.
Semantically this is an event handler. You are right, naming is questionable
(there is even an inspection that I had to suppress), we should come up with
something else probably.
As for resource injection, I think it has a lot of problems where we use it. It
is not intuitive, too verbose, triggers all sorts of compiler warnings
(uninitialized variable, etc). Explicit parameter injection is much easier to
use. For closures resource injection makes more sense due to serialization, but
here we don't serialize anything.
> .NET: Rename ILifecycleBean
> ---------------------------
>
> Key: IGNITE-3548
> URL: https://issues.apache.org/jira/browse/IGNITE-3548
> Project: Ignite
> Issue Type: Improvement
> Components: platforms
> Reporter: Pavel Tupitsyn
> Assignee: Pavel Tupitsyn
> Labels: .net, breaking-api
> Fix For: 2.0
>
>
> Bean is a Java term.
> Either rename this interface to something like ILifecycle, or rework the
> lifecycle notifications to C# event handlers.
> 2.0 migration guide has to be updated if needed:
> https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
