ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Лев Киселев <lev.kiselev.1...@gmail.com>
Subject Re: Contribution
Date Tue, 04 Feb 2020 15:09:06 GMT
Andrey, then I suggest firstly check if old annotations are present, and if
not - are there any new annotations. Do you agree?
And should I also replace usage of old annotations or leave it as it is?

вт, 4 февр. 2020 г. в 16:57, Andrey Gura <agura@apache.org>:

> Lev,
>
> > I'm confused about your first comment. Does it mean, that the only thing
> I
> can do is to create new annotation?
>
> Yes, it does. But in this case you can mark old annotations as
> deprecated because new annotations are real alternative.
>
> > So, I can't change implementation of *getDescription / getParameterName
> > (MBeanOperationInfo op, MBeanParameterInfo param, int seq)* methods from*
> > IgniteStandardMXBean* class,
> > and replace current annotations with new one.
>
> You have to support both old and new annotations due to a backward
> compatibility.
>
> On Tue, Feb 4, 2020 at 3:14 PM Лев Киселев <lev.kiselev.1769@gmail.com>
> wrote:
> >
> > Thanks for answer, Andrey.
> > I'm confused about your first comment. Does it mean, that the only thing
> I
> > can do is to create new annotation?
> > Because, as you said, I must leave unchanged all related to old
> annotations
> > functionality.
> > So, I can't change implementation of *getDescription / getParameterName
> > (MBeanOperationInfo op, MBeanParameterInfo param, int seq)* methods from*
> > IgniteStandardMXBean* class,
> > and replace current annotations with new one.
> >
> > вт, 4 февр. 2020 г. в 14:28, Andrey Gura <agura@apache.org>:
> >
> > > Hi, Lev!
> > >
> > > I think you understand task correctly.
> > >
> > > A couple of comments:
> > >
> > > - Existing annotations and related functionality must not be deleted
> > > because it is part of public API and we committed to have stable API
> > > between major releases.
> > > - I would suggest introduce @MXBeanParameter annotation instead of
> > > @MXBeanParameterInformation. Just shorten name.
> > >
> > > On Thu, Jan 30, 2020 at 8:53 PM Denis Magda <dmagda@apache.org> wrote:
> > > >
> > > > Nikolay, Andrey,
> > > >
> > > > Would you be the best committers to help out here? You are already
> > > deeply involved in metrics development and can quickly suggest Lev how
> to
> > > proceed with this task.
> > > >
> > > > -
> > > > Denis
> > > >
> > > >
> > > > On Thu, Jan 30, 2020 at 2:46 AM Лев Киселев <
> lev.kiselev.1769@gmail.com>
> > > wrote:
> > > >>
> > > >> Hello everyone, I have question about following task:
> > > >> [https://issues.apache.org/jira/browse/IGNITE-10698]
> > > >> Solution proposed in task description is seem to be logical.
> > > >> So, I need to every replace @MXBeanParametersNames and
> > > >> @MXBeanParametersDescriptions (everywhere, for uniformity) with
> > > something
> > > >> like:
> > > >> void methodName(@MXBeanParameterInformation(name = "name",
> description =
> > > >> "description") firstParameter, ...) {}.
> > > >> And, of course, need to change processing logic at
> > > >> getParameterName/getDescription methods from IgniteStandardMXBean.
> > > >> Do I understand correctly what needs to be done?
> > >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message