Request for review: @IgniteExperimental

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Request for review: @IgniteExperimental

Alexey Goncharuk
Igniters,

As discussed in the thread [1], we want to add an annotation marking an API
or feature experimental. The corresponding ticket and PR are [2],[3].

Please take a look at the description and suggest any edits to the javadoc
if necessary.

A side question - Anton Vinogradov asked me to mark the read-repair
feature as experimental in this PR. Should I also mark the IEP-35
interfaces there as well or should we move that change to a separate ticket?

[1]
http://apache-ignite-developers.2346864.n4.nabble.com/Internal-classes-are-exposed-in-public-API-td45146.html
[2] https://issues.apache.org/jira/browse/IGNITE-12559
[3] https://github.com/apache/ignite/pull/7279
Reply | Threaded
Open this post in threaded view
|

Re: Request for review: @IgniteExperimental

Alexey Goncharuk
Pavel, Igor,

Can you add similar annotations/attributes to platforms?

вт, 21 янв. 2020 г. в 16:01, Alexey Goncharuk <[hidden email]>:

> Igniters,
>
> As discussed in the thread [1], we want to add an annotation marking an
> API or feature experimental. The corresponding ticket and PR are [2],[3].
>
> Please take a look at the description and suggest any edits to the javadoc
> if necessary.
>
> A side question - Anton Vinogradov asked me to mark the read-repair
> feature as experimental in this PR. Should I also mark the IEP-35
> interfaces there as well or should we move that change to a separate ticket?
>
> [1]
> http://apache-ignite-developers.2346864.n4.nabble.com/Internal-classes-are-exposed-in-public-API-td45146.html
> [2] https://issues.apache.org/jira/browse/IGNITE-12559
> [3] https://github.com/apache/ignite/pull/7279
>
Reply | Threaded
Open this post in threaded view
|

Re: Request for review: @IgniteExperimental

Alexey Goncharuk
Thanks to all for the review, the annotation has been added, also created
an additional ticket for documentation.

вт, 21 янв. 2020 г. в 16:23, Alexey Goncharuk <[hidden email]>:

> Pavel, Igor,
>
> Can you add similar annotations/attributes to platforms?
>
> вт, 21 янв. 2020 г. в 16:01, Alexey Goncharuk <[hidden email]
> >:
>
>> Igniters,
>>
>> As discussed in the thread [1], we want to add an annotation marking an
>> API or feature experimental. The corresponding ticket and PR are [2],[3].
>>
>> Please take a look at the description and suggest any edits to the
>> javadoc if necessary.
>>
>> A side question - Anton Vinogradov asked me to mark the read-repair
>> feature as experimental in this PR. Should I also mark the IEP-35
>> interfaces there as well or should we move that change to a separate ticket?
>>
>> [1]
>> http://apache-ignite-developers.2346864.n4.nabble.com/Internal-classes-are-exposed-in-public-API-td45146.html
>> [2] https://issues.apache.org/jira/browse/IGNITE-12559
>> [3] https://github.com/apache/ignite/pull/7279
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: Request for review: @IgniteExperimental

Anton Vinogradov-2
Great, Thanks!

On Fri, Jan 24, 2020 at 1:14 PM Alexey Goncharuk <[hidden email]>
wrote:

> Thanks to all for the review, the annotation has been added, also created
> an additional ticket for documentation.
>
> вт, 21 янв. 2020 г. в 16:23, Alexey Goncharuk <[hidden email]
> >:
>
> > Pavel, Igor,
> >
> > Can you add similar annotations/attributes to platforms?
> >
> > вт, 21 янв. 2020 г. в 16:01, Alexey Goncharuk <
> [hidden email]
> > >:
> >
> >> Igniters,
> >>
> >> As discussed in the thread [1], we want to add an annotation marking an
> >> API or feature experimental. The corresponding ticket and PR are
> [2],[3].
> >>
> >> Please take a look at the description and suggest any edits to the
> >> javadoc if necessary.
> >>
> >> A side question - Anton Vinogradov asked me to mark the read-repair
> >> feature as experimental in this PR. Should I also mark the IEP-35
> >> interfaces there as well or should we move that change to a separate
> ticket?
> >>
> >> [1]
> >>
> http://apache-ignite-developers.2346864.n4.nabble.com/Internal-classes-are-exposed-in-public-API-td45146.html
> >> [2] https://issues.apache.org/jira/browse/IGNITE-12559
> >> [3] https://github.com/apache/ignite/pull/7279
> >>
> >
>