+1 for stability first
2014-12-16 16:50 GMT+01:00 Stephan Ewen <se...@apache.org>:
>
> I am currently working through the type hints pull request...
>
> Right now, I actually think that we should postpone this for the next
> bugfix release.
>
> This pull request changes actually some tricky mechanisms in the Java API
> (it delays the type information), which may have implications that me and
> Timo have not fully caught now.
>
> I don't feel Merging this last minute - this is prone to introduce errors.
>
> Any opinions on that?
>
>
>
> On Tue, Dec 16, 2014 at 3:36 PM, Stephan Ewen <se...@apache.org> wrote:
> >
> > +1 for adding the closure cleaner...
> > Am 16.12.2014 15:23 schrieb "Aljoscha Krettek" <aljos...@apache.org>:
> >
> > Do we want the Java Closure Cleaner in 0.8 as well?
> >>
> >> On Tue, Dec 16, 2014 at 10:30 AM, Márton Balassi
> >> <balassi.mar...@gmail.com> wrote:
> >> > Cool, thanks.
> >> >
> >> > On Tue, Dec 16, 2014 at 10:29 AM, Stephan Ewen <se...@apache.org>
> >> wrote:
> >> >>
> >> >> Hi!
> >> >>
> >> >> The type hints PR is ready now, Timo made some final adjustments
> >> yesterday
> >> >> evening. I'll review and merge it now, then we can fork the branch.
> >> >>
> >> >> Cheers,
> >> >> Stephan
> >> >>
> >> >>
> >> >> On Mon, Dec 15, 2014 at 10:08 PM, Márton Balassi <
> >> balassi.mar...@gmail.com
> >> >> >
> >> >> wrote:
> >> >> >
> >> >> > Hey,
> >> >> >
> >> >> > Checking the PRs listed by Stephan it seems every import one has
> been
> >> >> > merged. As for the optional ones the following two are still
> pending:
> >> >> >
> >> >> >
> >> >> > Add support for Subclasses, Interfaces, Abstract Classes as POJOs
> >> >> >
> >> >> > [FLINK-1245] Introduce TypeHints for Java API operators
> >> >> >
> >> >> >
> >> >> > Robert +1'd the TypeHints PR and Timo has just updated it. Should
> we
> >> wait
> >> >> > for it?
> >> >> >
> >> >> > Cheers,
> >> >> >
> >> >> > Marton
> >> >> >
> >> >> > On Fri, Dec 12, 2014 at 2:29 PM, Ufuk Celebi <u...@apache.org>
> wrote:
> >> >> > >
> >> >> > > On Wed, Dec 10, 2014 at 7:35 PM, Stephan Ewen <se...@apache.org>
> >> >> wrote:
> >> >> > > >
> >> >> > > > ----------------------------------------------
> >> >> > > > Important Pull Requests to merge:
> >> >> > > > ----------------------------------------------
> >> >> > > >
> >> >> > > > [FLINK-1287] LocalizableSplitAssigner prefers splits with less
> >> >> degrees
> >> >> > of
> >> >> > > > freedom
> >> >> > > >
> >> >> > > > [FLINK-1305] [FLINK-1304] Test for HadoopInputWrapper and
> >> >> NullWritable
> >> >> > > > support
> >> >> > > >
> >> >> > > > Upgraded HBase addon to HBase 0.98.x and new Tuple APIs + fix
> of
> >> >> > > > ExecutionEnvironment
> >> >> > > > (this is partially merged, there are a few POM entries missing
> >> for
> >> >> > > Hadoop 1
> >> >> > > > support)
> >> >> > > >
> >> >> > >
> >> >> > > The first two PRs are still open. I will have a look at them as
> >> well
> >> >> > > (Robert already tested one).
> >> >> > >
> >> >> > > @Robert:
> >> >> > > Regarding the Hbase PR: Is it resolved by 1b69749 ("Enable
> >> flink-hbase
> >> >> > > module for hadoop1 build as well")?
> >> >> > >
> >> >> > >
> >> >> > > It would be nice if had a look at the remaining important ones
> >> today.
> >> >> :-)
> >> >> > >
> >> >> >
> >> >>
> >>
> >
>
