[ 
https://issues.apache.org/jira/browse/IGNITE-3548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15962955#comment-15962955
 ] 

Vladimir Ozerov commented on IGNITE-3548:
-----------------------------------------

[~ptupitsyn],
This is very questionable change, I would close the ticket as "Won't Fix" 
instead. The main problem is that we named something as "EventHandler", but 
this is not real {{EventHandler}} delegate as .NET users might expect. 
Changes to method signature is not obvious either. Lifecycle beans are stateful 
instances. As such, resource injection appears to be more reasonable to me than 
passing values through method call.

> .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)

Reply via email to