Necessity to have MBeanServerResource annotation

[Denis Magda]

Igniters,

There is MBeanServerResource annotation available as a part of Ignite.
However, according to this JIRA issue [1] its never processed at the
kernel level.

My understanding is that we should neither delete it nor deprecate but
rather support at the kernel level injecting MBeanServer when the latter
is specified in IgniteConfiguration.

Any thoughts on this?

[1] https://issues.apache.org/jira/browse/IGNITE-1800

[yzhdanov]

Guys, I don't think we should delete this since it is not hard at all to
fix injection properly.

--Yakov

2015-11-25 11:40 GMT+03:00 Denis Magda <[hidden email]>:

> Igniters,
>
> There is MBeanServerResource annotation available as a part of Ignite.
> However, according to this JIRA issue [1] its never processed at the
> kernel level.
>
> My understanding is that we should neither delete it nor deprecate but
> rather support at the kernel level injecting MBeanServer when the latter is
> specified in IgniteConfiguration.
>
> Any thoughts on this?
>
> [1] https://issues.apache.org/jira/browse/IGNITE-1800
>

[Artem Shutak]

Igniters,

As I remember, support of the annotation has been deleted intentionally. I
think Nik can shed light on this.

By the way, currently, you can use IgniteInstanceResource annotation to get
Ignite instance and getting MBeanServer.

ignite.configuration().getMBeanServer()

Thanks,
-- Artem --

On Wed, Nov 25, 2015 at 12:19 PM, Yakov Zhdanov <[hidden email]> wrote:

> Guys, I don't think we should delete this since it is not hard at all to
> fix injection properly.
>
> --Yakov
>
> 2015-11-25 11:40 GMT+03:00 Denis Magda <[hidden email]>:
>
> > Igniters,
> >
> > There is MBeanServerResource annotation available as a part of Ignite.
> > However, according to this JIRA issue [1] its never processed at the
> > kernel level.
> >
> > My understanding is that we should neither delete it nor deprecate but
> > rather support at the kernel level injecting MBeanServer when the latter
> is
> > specified in IgniteConfiguration.
> >
> > Any thoughts on this?
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-1800
> >
>

[yzhdanov]

Thanks for clarifications, Artem!

--Yakov

2015-11-25 12:27 GMT+03:00 Artem Shutak <[hidden email]>:

> Igniters,
>
> As I remember, support of the annotation has been deleted intentionally. I
> think Nik can shed light on this.
>
> By the way, currently, you can use IgniteInstanceResource annotation to get
> Ignite instance and getting MBeanServer.
>
> ignite.configuration().getMBeanServer()
>
> Thanks,
> -- Artem --
>
> On Wed, Nov 25, 2015 at 12:19 PM, Yakov Zhdanov <[hidden email]>
> wrote:
>
> > Guys, I don't think we should delete this since it is not hard at all to
> > fix injection properly.
> >
> > --Yakov
> >
> > 2015-11-25 11:40 GMT+03:00 Denis Magda <[hidden email]>:
> >
> > > Igniters,
> > >
> > > There is MBeanServerResource annotation available as a part of Ignite.
> > > However, according to this JIRA issue [1] its never processed at the
> > > kernel level.
> > >
> > > My understanding is that we should neither delete it nor deprecate but
> > > rather support at the kernel level injecting MBeanServer when the
> latter
> > is
> > > specified in IgniteConfiguration.
> > >
> > > Any thoughts on this?
> > >
> > > [1] https://issues.apache.org/jira/browse/IGNITE-1800
> > >
> >
>

[Nikolay Tikhonov]

Artem right. The annotation should be removed in
https://issues.apache.org/jira/browse/IGNITE-35 ticket.

On Wed, Nov 25, 2015 at 12:34 PM, Yakov Zhdanov <[hidden email]> wrote:

> Thanks for clarifications, Artem!
>
> --Yakov
>
> 2015-11-25 12:27 GMT+03:00 Artem Shutak <[hidden email]>:
>
> > Igniters,
> >
> > As I remember, support of the annotation has been deleted intentionally.
> I
> > think Nik can shed light on this.
> >
> > By the way, currently, you can use IgniteInstanceResource annotation to
> get
> > Ignite instance and getting MBeanServer.
> >
> > ignite.configuration().getMBeanServer()
> >
> > Thanks,
> > -- Artem --
> >
> > On Wed, Nov 25, 2015 at 12:19 PM, Yakov Zhdanov <[hidden email]>
> > wrote:
> >
> > > Guys, I don't think we should delete this since it is not hard at all
> to
> > > fix injection properly.
> > >
> > > --Yakov
> > >
> > > 2015-11-25 11:40 GMT+03:00 Denis Magda <[hidden email]>:
> > >
> > > > Igniters,
> > > >
> > > > There is MBeanServerResource annotation available as a part of
> Ignite.
> > > > However, according to this JIRA issue [1] its never processed at the
> > > > kernel level.
> > > >
> > > > My understanding is that we should neither delete it nor deprecate
> but
> > > > rather support at the kernel level injecting MBeanServer when the
> > latter
> > > is
> > > > specified in IgniteConfiguration.
> > > >
> > > > Any thoughts on this?
> > > >
> > > > [1] https://issues.apache.org/jira/browse/IGNITE-1800
> > > >
> > >
> >
>

[Denis Magda]

Got all of you thanks.

Merged the changes from

https://issues.apache.org/jira/browse/IGNITE-1800


Roman, thanks for the contribution!

--
Denis

On 11/25/2015 3:55 PM, Nikolay Tikhonov wrote:

> Artem right. The annotation should be removed in
> https://issues.apache.org/jira/browse/IGNITE-35 ticket.
>
> On Wed, Nov 25, 2015 at 12:34 PM, Yakov Zhdanov <[hidden email]> wrote:
>
>> Thanks for clarifications, Artem!
>>
>> --Yakov
>>
>> 2015-11-25 12:27 GMT+03:00 Artem Shutak <[hidden email]>:
>>
>>> Igniters,
>>>
>>> As I remember, support of the annotation has been deleted intentionally.
>> I
>>> think Nik can shed light on this.
>>>
>>> By the way, currently, you can use IgniteInstanceResource annotation to
>> get
>>> Ignite instance and getting MBeanServer.
>>>
>>> ignite.configuration().getMBeanServer()
>>>
>>> Thanks,
>>> -- Artem --
>>>
>>> On Wed, Nov 25, 2015 at 12:19 PM, Yakov Zhdanov <[hidden email]>
>>> wrote:
>>>
>>>> Guys, I don't think we should delete this since it is not hard at all
>> to
>>>> fix injection properly.
>>>>
>>>> --Yakov
>>>>
>>>> 2015-11-25 11:40 GMT+03:00 Denis Magda <[hidden email]>:
>>>>
>>>>> Igniters,
>>>>>
>>>>> There is MBeanServerResource annotation available as a part of
>> Ignite.
>>>>> However, according to this JIRA issue [1] its never processed at the
>>>>> kernel level.
>>>>>
>>>>> My understanding is that we should neither delete it nor deprecate
>> but
>>>>> rather support at the kernel level injecting MBeanServer when the
>>> latter
>>>> is
>>>>> specified in IgniteConfiguration.
>>>>>
>>>>> Any thoughts on this?
>>>>>
>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-1800
>>>>>