flink-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Theodore Vasiloudis <theodoros.vasilou...@gmail.com>
Subject Re: [DISCUSS] @Public libraries
Date Wed, 23 Nov 2016 16:21:39 GMT
What Till said is true for FlinkML, until all the moving parts are in place
there's not much point in annotating any as Public. The Spark project has
the @Experimental tag IIRC, that would fit our case better.

On Wed, Nov 23, 2016 at 4:09 PM, Till Rohrmann <trohrmann@apache.org> wrote:

> I think in general annotating library methods/classes is a good idea. The
> question is just which APIs are going to be marked stable.
>
> In the past we've seen that we might have marked some of Flink's APIs
> stable too early. As a consequence we have to carry them along for quite
> some time (at the very least until Flink 2.0).
>
> If we now come to the conclusion that the library APIs are not yet stable
> enough to mark them Public, then we will probably mark a lot of the API
> PublicEvolving which adds only little value for the user.
>
> Cheers,
> Till
>
> On Wed, Nov 23, 2016 at 2:25 PM, Aljoscha Krettek <aljoscha@apache.org>
> wrote:
>
> > I would be for also annotating library methods/classes. Maybe Robert has
> a
> > stronger opinion on this because he introduced these annotations.
> >
> > On Tue, 22 Nov 2016 at 18:56 Greg Hogan <code@greghogan.com> wrote:
> >
> > > Hi all,
> > >
> > > Should stable APIs in Flink's CEP, ML, and Gelly libraries be annotated
> > > @Public or restricted to use of @PublicEvolving?
> > >
> > > We would ensure that library APIs do not add restrictions to the core
> > APIs.
> > > Libraries could use @PublicEvolving or @Internal core APIs within
> @Public
> > > or @PublicEvolving components as long as the functionality could be
> > adapted
> > > or rewritten as necessary. An example: CombineHint is @Internal. Gelly
> > > could use CombineHint in a @Public method but could not accept
> > CombineHint
> > > as a parameter to a @Public method.
> > >
> > > Greg
> > >
> >
>

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