I am fine with it since it only touches a utility class. So +1 to include it as well.
– Ufuk On Mon, Apr 18, 2016 at 12:07 PM, Fabian Hueske <fhue...@gmail.com> wrote: > There is also FLINK-3657 which was recently merged to master. > This feature was requested by the Mahout community and the commit changes > the visibility of a method in DataSetUtils. > So strictly speaking, this is not a bug fix but a new feature. On the other > hand, it is very lightweight change and does not influence anything else. > > I would be in favor of adding this change to Flink 1.0.2 as well. > What do others think? > > Cheers, Fabian > > 2016-04-18 11:52 GMT+02:00 Ufuk Celebi <u...@apache.org>: > >> This vote is cancelled in favour of RC3, because of the problem raised >> by Fabian. >> >> On Fri, Apr 15, 2016 at 1:02 PM, Fabian Hueske <fhue...@gmail.com> wrote: >> > There is a request from the Mahout community to include a fix for >> > FLINK-3762 in 1.0.2. >> > Stephan gave a +1 [1] and I would also like to include it. >> > >> > I'm running tests right now and will merge the fix they provided later >> > today. >> > >> > Cheers, Fabian >> > >> > [1] >> > >> https://mail-archives.apache.org/mod_mbox/flink-dev/201604.mbox/%3CCANC1h_v2TwooQnXsfgYhpPCremT37%2BOwGjz_d5qsLdE0D%2BfmDA%40mail.gmail.com%3E >> > >> > >> > 2016-04-15 12:47 GMT+02:00 Ufuk Celebi <u...@apache.org>: >> > >> >> Dear Flink community, >> >> >> >> Please vote on releasing the following candidate as Apache Flink version >> >> 1.0.2. >> >> >> >> The commit to be voted on: >> >> 058cc5de789331983830ce4720cd98c3af786109 >> >> >> >> Branch: >> >> release-1.0.2-rc2 (see >> >> >> >> >> https://git1-us-west.apache.org/repos/asf/flink/?p=flink.git;a=shortlog;h=refs/heads/release-1.0.2-rc2 >> >> ) >> >> >> >> The release artifacts to be voted on can be found at: >> >> http://home.apache.org/~uce/flink-1.0.2-rc2/ >> >> >> >> The release artifacts are signed with the key with fingerprint 9D403309: >> >> http://www.apache.org/dist/flink/KEYS >> >> >> >> The staging repository for this release can be found at: >> >> https://repository.apache.org/content/repositories/orgapacheflink-1091 >> >> >> >> ------------------------------------------------------------- >> >> >> >> The vote is open for the next 72 hours and passes if a majority of at >> least >> >> three +1 PMC votes are cast. >> >> >> >> The vote ends on Tuesday April 19, 2016 (counting the weekend as a >> single >> >> day). >> >> >> >> [ ] +1 Release this package as Apache Flink 1.0.2 >> >> [ ] -1 Do not release this package because ... >> >> >> >> =================================== >> >> >> >> The following commits have been added since the 1.0.1 release (excluding >> >> docs), >> >> most notably a performance optimization for the RocksDB state backend >> and >> >> a fix >> >> for proper passing of dynamic YARN properties to the Client. >> >> >> >> * aadc5fa - [FLINK-3760] Fix StateDescriptor.readObject (19 hours ago) >> >> <Aljoscha Krettek> >> >> * ea50ed3 - [FLINK-3716] [kafka consumer] Decreasing socket timeout so >> >> testFailOnNoBroker() will pass before JUnit timeout (29 hours ago) >> >> <Todd Lisonbee> >> >> * ff38202 - [FLINK-3730] Fix RocksDB Local Directory Initialization >> >> (29 hours ago) <Aljoscha Krettek> >> >> * 4f9c198 - [FLINK-3712] Make all dynamic properties available to the >> >> CLI frontend (3 days ago) <Robert Metzger> >> >> * 1554c9b - [FLINK-3688] WindowOperator.trigger() does not emit >> >> Watermark anymore (6 days ago) <Konstantin Knauf> >> >> * 43093e3 - [FLINK-3697] Properly access type information for nested >> >> POJO key selection (6 days ago) <Robert Metzger> >> >> * 17909aa - [FLINK-3654] Disable Write-Ahead-Log in RocksDB State (7 >> >> days ago) <Aljoscha Krettek> >> >> * e0dc5c1 - [FLINK-3595] [runtime] Eagerly destroy buffer pools on >> >> cancelling (10 days ago) <Ufuk Celebi> >> >> >>
