I've worked on adding Kryo support to Flink today. I would really like to
include this into the release. Many users were complaining about this (
https://issues.apache.org/jira/browse/FLINK-610).
I've based my work on top of Timo's pull request. If we're not going to
merge it, I will have to rebase my code to the current master. ... I'll try
and see if its easily doable.
On Tue, Dec 16, 2014 at 5:02 PM, Stephan Ewen <se...@apache.org> wrote:
>
> 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.
> > > >> >> :-)
> > > >> >> > >
> > > >> >> >
> > > >> >>
> > > >>
> > > >
> > >
> >
>
