ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Ozerov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-3548) .NET: Rename ILifecycleBean
Date Mon, 10 Apr 2017 14:29:41 GMT

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

Mime
View raw message