Re: Releasing Flink 1.1.3

2016-10-07 Thread Kostas Kloudas
Hi all, Tried the Kafka source and the continuous file sources and everything seems to be working fine. Kostas > On Oct 5, 2016, at 6:53 PM, Ufuk Celebi wrote: > > Thanks for your effort! > > I'm going to kick off the build now. :-) > > On Wed, Oct 5, 2016 at 5:59 PM, Maximilian Michels wro

Re: Releasing Flink 1.1.3

2016-10-05 Thread Ufuk Celebi
Thanks for your effort! I'm going to kick off the build now. :-) On Wed, Oct 5, 2016 at 5:59 PM, Maximilian Michels wrote: > Kostas PR https://github.com/apache/flink/pull/2593 is merged. I think > we're good to go. > > On Wed, Oct 5, 2016 at 3:44 PM, Maximilian Michels wrote: >> I'll merge htt

Re: Releasing Flink 1.1.3

2016-10-05 Thread Maximilian Michels
Kostas PR https://github.com/apache/flink/pull/2593 is merged. I think we're good to go. On Wed, Oct 5, 2016 at 3:44 PM, Maximilian Michels wrote: > I'll merge https://github.com/apache/flink/pull/2548 for the release. > It's cosmetic but it avoids a NPE in case the user jar doesn't contain > Fli

Re: Releasing Flink 1.1.3

2016-10-05 Thread Maximilian Michels
I'll merge https://github.com/apache/flink/pull/2548 for the release. It's cosmetic but it avoids a NPE in case the user jar doesn't contain Flink jobs. On Wed, Oct 5, 2016 at 12:59 PM, Kostas Kloudas wrote: > Hi Ufuk, > > Thanks for being the release manager. > > There is already an open PR unde

Re: Releasing Flink 1.1.3

2016-10-05 Thread Kostas Kloudas
Hi Ufuk, Thanks for being the release manager. There is already an open PR under review for the Continuous File Sources. Kostas > On Oct 4, 2016, at 11:25 PM, Ufuk Celebi wrote: > > Thanks for your prompt responses. > > Except for the streaming file source issues, all mentioned issues are >

Re: Releasing Flink 1.1.3

2016-10-04 Thread Ufuk Celebi
Thanks for your prompt responses. Except for the streaming file source issues, all mentioned issues are addressed. As soon as the last one is in, I can kick off the first RC. @Kostas, Stephan: you didn't mention anything here or in the PR. Did you have time to work on this? :) – Ufuk On Tue, Oc

Re: Releasing Flink 1.1.3

2016-10-04 Thread Stephan Ewen
Concerning the Pull Requests about Kafka 0.8 offset committing: They look good, but I would actually like to not merge them to Flink 1.1.3, as they may result in slightly changed behavior for users of Kafka 0.8 The crucial fix (Kafka 0.9) is already in, so on the Kafka side are are good to go, in

Re: Releasing Flink 1.1.3

2016-10-04 Thread Tzu-Li (Gordon) Tai
Thanks Ufuk for stepping up as RM! Regarding whether FLINK-4723 / FLINK-4727 should be included: The failing test on PR #2580 is unrelated to the change. However, I think it’s reasonable to skip them if we’re aiming to get a RC out today, as they’ll need more time for reviewing. Not including the

Re: Releasing Flink 1.1.3

2016-10-04 Thread Fabian Hueske
Thanks Ufuk for stepping up as release manager! Yes, I will backport the fix for FLINK-4311 to Flink 1.1.3 and merge it today. 2016-10-04 12:07 GMT+02:00 Ufuk Celebi : > If there are no objections I would like to be the release manager for > this release. > > Futhermore, I would like to add FLIN

Re: Releasing Flink 1.1.3

2016-10-04 Thread Márton Balassi
Thanks for volunteering for RM, Ufuk. On Tue, Oct 4, 2016 at 12:07 PM, Ufuk Celebi wrote: > If there are no objections I would like to be the release manager for > this release. > > Futhermore, I would like to add FLINK-4732 (Maven junction plugin > security issue) to the list of fixes for this

Re: Releasing Flink 1.1.3

2016-10-04 Thread Ufuk Celebi
If there are no objections I would like to be the release manager for this release. Futhermore, I would like to add FLINK-4732 (Maven junction plugin security issue) to the list of fixes for this release. Other than that, I think the list in this thread is good and we should now focus on getting a

Re: Releasing Flink 1.1.3

2016-10-04 Thread Tzu-Li (Gordon) Tai
Update: FLINK-4618 (https://github.com/apache/flink/pull/2579) had been merged. There’s another 2 follow-up issues that I also want to include in 1.1.3 for the Kafka connector after review, if others agree: - FLINK-4723 (https://github.com/apache/flink/pull/2580) - FLINK-4727 (https://github.com

Re: Releasing Flink 1.1.3

2016-09-29 Thread Tzu-Li (Gordon) Tai
+1. We should also include FLINK-4618. There isn’t a PR for it yet, but the fix should be fairly quick. Gordon On September 30, 2016 at 1:50:45 AM, Stephan Ewen (se...@apache.org) wrote: +1 for including - FLINK-4311 - FLINK-4702 (https://github.com/apache/flink/pull/2574) - FLINK-4329 (h

Re: Releasing Flink 1.1.3

2016-09-29 Thread Stephan Ewen
+1 for including - FLINK-4311 - FLINK-4702 (https://github.com/apache/flink/pull/2574) - FLINK-4329 (https://github.com/apache/flink/pull/2546) On Thu, Sep 29, 2016 at 3:38 PM, Ufuk Celebi wrote: > +1... Personally, I'm a big fan of frequent bugfix releases :-) > > A user already report

Re: Releasing Flink 1.1.3

2016-09-29 Thread Ufuk Celebi
+1... Personally, I'm a big fan of frequent bugfix releases :-) A user already reported running into the file source problem and having a very hard time debugging it. This would be good to fix asap. – Ufuk On Thu, Sep 29, 2016 at 3:20 PM, Fabian Hueske wrote: > Sounds good to me. > > Can we inc

Re: Releasing Flink 1.1.3

2016-09-29 Thread Fabian Hueske
Sounds good to me. Can we include a fix for FLINK-4311 as well? There is a PR [1] which is good to go IMO modulo resetting the updated HBase dependency for the 1.1.3 fix. For 1.2.0 we should start a discussion about bumping the version. Cheers, Fabian [1] https://github.com/apache/flink/pull/233

Releasing Flink 1.1.3

2016-09-29 Thread Stephan Ewen
Hi all! I wanted to bring up having a Flink 1.1.3 bugfix release in the near future. There are some fixed right now in the release-1.1 branch already (merging windows, optimizer union, ...) and there is a major bug that is currently fixed in the master: https://issues.apache.org/jira/browse/FLINK