Yes, one thing I just realized is that the streaming API is not aware of
MissingTypeInfos and it may cause weird behavior.
On Tue, Dec 16, 2014 at 4:56 PM, Fabian Hueske <fhue...@gmail.com> wrote:
>
> +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.
> > >> >> :-)
> > >> >> > >
> > >> >> >
> > >> >>
> > >>
> > >
> >
>
