+1

- Staged Apache source & binary convenience releases looks good
- Built from source (macOS w/ Scala 2.11, hadoop-free, hadoop-2.8)
- Locally tested topic regex subscription for the Kafka consumer
- Quickstart projects looks good

Other things verified that are carried from previous RC votes
- Cluster tests on AWS with configuration detailed in [1], with special
focus on dynamic Kafka partition discovery
- Kinesis connector, Elasticsearch connector runs fine with cluster
execution + locally in IDE, without any dependency clashes

[1]
https://docs.google.com/document/d/1cOkycJwEKVjG_onnpl3bQNTq7uebh48zDtIJxceyU2E/edit#heading=h.sintcv4ccegd

On Mon, Dec 11, 2017 at 9:28 PM, Timo Walther <twal...@apache.org> wrote:

> +1 (binding)
>
> - build the source locally
> - run various table programs
> - checked the resource consumption of table programs with retention
> enabled and disabled
> - built a quickstart project
> - tested the web ui submission (found https://issues.apache.org/jira
> /browse/FLINK-8187 but this is non-blocking)
>
>
> Am 12/11/17 um 2:16 PM schrieb Chesnay Schepler:
>
> +1 (binding)
>>
>> - checked contents of flink-dist for unshaded dependencies
>> - ran python examples (with/-out arguments) locally
>> - ran jobs on yarn on a cluster testing optional hadoop dependency
>> - verified that quickstarts work
>> - checked JM/TM logs for anything suspicious
>>
>> On 11.12.2017 11:29, Fabian Hueske wrote:
>>
>>> +1 (binding)
>>>
>>> - Checked hashes & signatures
>>> - Checked no binaries in source release
>>> - Checked Flink version in Quickstart pom files
>>>
>>> Cheers, Fabian
>>>
>>> 2017-12-11 11:26 GMT+01:00 Stefan Richter <s.rich...@data-artisans.com>:
>>>
>>> +1 (non-binding)
>>>>
>>>> - did extensive cluster tests on Google Cloud with special focus on
>>>> checkpointing and recovery and Kafka 0.11 end-to-end exactly-once +
>>>> at-least-once.
>>>> - build from source.
>>>>
>>>> Am 11.12.2017 um 09:53 schrieb Piotr Nowojski <pi...@data-artisans.com
>>>>> >:
>>>>>
>>>>> Hi,
>>>>>
>>>>> +1 (non-binding)
>>>>>
>>>>> I have:
>>>>> - verified Scala and Java sample projects are creating and working
>>>>>
>>>> properly and that Quickstart docs are ok
>>>>
>>>>> - verified that ChildFirstClassloader allows user to run his
>>>>> application
>>>>>
>>>> with some custom akka version
>>>>
>>>>> - tested Kafka 0.11 end to end exactly once
>>>>> - did some manual checks whether docs/distribution files are ok
>>>>>
>>>>> Piotrek
>>>>>
>>>>> On 8 Dec 2017, at 16:49, Stephan Ewen <se...@apache.org> wrote:
>>>>>>
>>>>>> @Eron Given that this is actually an undocumented "internal" feature
>>>>>> at
>>>>>> this point, I would not expect that it is used heavily beyond Pravega.
>>>>>>
>>>>>> Unless you feel strongly that this is a major issue, I would go ahead
>>>>>>
>>>>> with
>>>>
>>>>> the release...
>>>>>>
>>>>>> On Fri, Dec 8, 2017 at 3:18 PM, Aljoscha Krettek <aljos...@apache.org
>>>>>> >
>>>>>> wrote:
>>>>>>
>>>>>> Thanks for the update! I would also say it's not a blocker but we
>>>>>>>
>>>>>> should
>>>>
>>>>> make sure that we don't break this after 1.4, then.
>>>>>>>
>>>>>>> On 7. Dec 2017, at 22:37, Eron Wright <eronwri...@gmail.com> wrote:
>>>>>>>>
>>>>>>>> Just discovered:  the removal of Flink's Future (FLINK-7252) causes
>>>>>>>> a
>>>>>>>> breaking change in connectors that use
>>>>>>>> `org.apache.flink.runtime.checkpoint.MasterTriggerRestoreHook`,
>>>>>>>>
>>>>>>> because
>>>>
>>>>> `Future` is a type on one of the methods.
>>>>>>>>
>>>>>>>> To my knowledge, this affects only the Pravega connector.  Curious
>>>>>>>> to
>>>>>>>>
>>>>>>> know
>>>>>>>
>>>>>>>> whether any other connectors are affected.  I don't think we (Dell
>>>>>>>>
>>>>>>> EMC)
>>>>
>>>>> consider it a blocker but it will mean that the connector is Flink
>>>>>>>>
>>>>>>> 1.4+.
>>>>
>>>>> Eron
>>>>>>>>
>>>>>>>>
>>>>>>>> On Thu, Dec 7, 2017 at 12:25 PM, Aljoscha Krettek <
>>>>>>>>
>>>>>>> aljos...@apache.org>
>>>>
>>>>> wrote:
>>>>>>>>
>>>>>>>> I just noticed that I did a copy-and-paste error and the last
>>>>>>>>>
>>>>>>>> paragraph
>>>>
>>>>> about voting period should be this:
>>>>>>>>>
>>>>>>>>> The vote will be open for at least 72 hours. It is adopted by
>>>>>>>>>
>>>>>>>> majority
>>>>
>>>>> approval, with at least 3 PMC affirmative votes.
>>>>>>>>>
>>>>>>>>> Best,
>>>>>>>>> Aljoscha
>>>>>>>>>
>>>>>>>>> On 7. Dec 2017, at 19:24, Bowen Li <bowen...@offerupnow.com>
>>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>> I agree that it shouldn't block the release. The doc website part
>>>>>>>>>> is
>>>>>>>>>>
>>>>>>>>> even
>>>>>>>
>>>>>>>> better!
>>>>>>>>>>
>>>>>>>>>> On Thu, Dec 7, 2017 at 1:09 AM, Aljoscha Krettek <
>>>>>>>>>>
>>>>>>>>> aljos...@apache.org>
>>>>
>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>> Good catch, yes. This shouldn't block the release, though, since
>>>>>>>>>>>
>>>>>>>>>> the
>>>>
>>>>> doc
>>>>>>>
>>>>>>>> is always built form the latest state of a release branch, i.e. the
>>>>>>>>>>>
>>>>>>>>>> 1.4
>>>>>>>
>>>>>>>> doc
>>>>>>>>>
>>>>>>>>>> on the website will update as soon as the doc on the release-1.4
>>>>>>>>>>>
>>>>>>>>>> branch
>>>>>>>
>>>>>>>> is
>>>>>>>>>
>>>>>>>>>> updated.
>>>>>>>>>>>
>>>>>>>>>>> On 6. Dec 2017, at 20:47, Bowen Li <bowen...@offerupnow.com>
>>>>>>>>>>>>
>>>>>>>>>>> wrote:
>>>>
>>>>> Hi Aljoscha,
>>>>>>>>>>>>
>>>>>>>>>>>> I found Flink's State doc and javaDoc are very ambiguous on what
>>>>>>>>>>>>
>>>>>>>>>>> the
>>>>
>>>>> replacement of FoldingState is, which will confuse a lot of
>>>>>>>>>>>>
>>>>>>>>>>> users. We
>>>>
>>>>> need
>>>>>>>>>>>
>>>>>>>>>>>> to fix it in 1.4 release.
>>>>>>>>>>>>
>>>>>>>>>>>> I have submitted a PR at https://github.com/apache/
>>>>>>>>>>>>
>>>>>>>>>>> flink/pull/5129
>>>>
>>>>> Thanks,
>>>>>>>>>>>> Bowen
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Dec 6, 2017 at 5:56 AM, Aljoscha Krettek <
>>>>>>>>>>>>
>>>>>>>>>>> aljos...@apache.org>
>>>>>>>
>>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> Hi everyone,
>>>>>>>>>>>>>
>>>>>>>>>>>>> Please review and vote on release candidate #3 for the version
>>>>>>>>>>>>>
>>>>>>>>>>>> 1.4.0,
>>>>>>>
>>>>>>>> as
>>>>>>>>>
>>>>>>>>>> follows:
>>>>>>>>>>>>> [ ] +1, Approve the release
>>>>>>>>>>>>> [ ] -1, Do not approve the release (please provide specific
>>>>>>>>>>>>>
>>>>>>>>>>>> comments)
>>>>>>>
>>>>>>>>
>>>>>>>>>>>>> The complete staging area is available for your review, which
>>>>>>>>>>>>>
>>>>>>>>>>>> includes:
>>>>>>>>>
>>>>>>>>>> * JIRA release notes [1],
>>>>>>>>>>>>> * the official Apache source release and binary convenience
>>>>>>>>>>>>>
>>>>>>>>>>>> releases
>>>>
>>>>> to
>>>>>>>>>
>>>>>>>>>> be
>>>>>>>>>>>
>>>>>>>>>>>> deployed to dist.apache.org[2], which are signed with the key
>>>>>>>>>>>>>
>>>>>>>>>>>> with
>>>>
>>>>> fingerprint F2A67A8047499BBB3908D17AA8F4FD97121D7293 [3],
>>>>>>>>>>>>> * all artifacts to be deployed to the Maven Central Repository
>>>>>>>>>>>>>
>>>>>>>>>>>> [4],
>>>>
>>>>> * source code tag "release-1.4.0-rc1" [5],
>>>>>>>>>>>>> * website pull request listing the new release [6].
>>>>>>>>>>>>>
>>>>>>>>>>>>> Please have a careful look at the website PR because I changed
>>>>>>>>>>>>>
>>>>>>>>>>>> some
>>>>
>>>>> wording and we're now also releasing a binary without Hadoop
>>>>>>>>>>>>>
>>>>>>>>>>>> dependencies.
>>>>>>>>>>>
>>>>>>>>>>>> Please use this document for coordinating testing efforts: [7]
>>>>>>>>>>>>>
>>>>>>>>>>>>> The only change between RC1 and this RC2 is that the source
>>>>>>>>>>>>>
>>>>>>>>>>>> release
>>>>
>>>>> package does not include the erroneously included binary Ruby
>>>>>>>>>>>>>
>>>>>>>>>>>> dependencies
>>>>>>>>>>>
>>>>>>>>>>>> of the documentation anymore. Because of this I would like to
>>>>>>>>>>>>>
>>>>>>>>>>>> propose
>>>>>>>
>>>>>>>> a
>>>>>>>>>
>>>>>>>>>> shorter voting time and close the vote around the time that RC1
>>>>>>>>>>>>>
>>>>>>>>>>>> would
>>>>>>>
>>>>>>>> have
>>>>>>>>>>>
>>>>>>>>>>>> closed. This would mean closing by end of Wednesday. Please let
>>>>>>>>>>>>>
>>>>>>>>>>>> me
>>>>
>>>>> know
>>>>>>>>>
>>>>>>>>>> if
>>>>>>>>>>>
>>>>>>>>>>>> you disagree with this. The vote is adopted by majority
>>>>>>>>>>>>> approval,
>>>>>>>>>>>>>
>>>>>>>>>>>> with
>>>>>>>
>>>>>>>> at
>>>>>>>>>>>
>>>>>>>>>>>> least 3 PMC affirmative votes.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Thanks,
>>>>>>>>>>>>> Your friendly Release Manager
>>>>>>>>>>>>>
>>>>>>>>>>>>> [1] https://issues.apache.org/jira/secure/ReleaseNote.jspa?
>>>>>>>>>>>>> projectId=12315522&version=12340533
>>>>>>>>>>>>> [2] http://people.apache.org/~aljoscha/flink-1.4.0-rc3/
>>>>>>>>>>>>> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
>>>>>>>>>>>>> [4] https://repository.apache.org/content/repositories/
>>>>>>>>>>>>>
>>>>>>>>>>>> orgapacheflink-1141
>>>>>>>>>>>
>>>>>>>>>>>> [5] https://git-wip-us.apache.org/
>>>>>>>>>>>>> repos/asf?p=flink.git;a=tag;h=
>>>>>>>>>>>>> 8fb9635dd2e64dbb20887c84f646f02034b57cb1
>>>>>>>>>>>>> [6] https://github.com/apache/flink-web/pull/95
>>>>>>>>>>>>> [7] https://docs.google.com/document/d/1cOkycJwEKVjG_
>>>>>>>>>>>>> onnpl3bQNTq7uebh48zDtIJxceyU2E/edit?usp=sharing
>>>>>>>>>>>>>
>>>>>>>>>>>>> Pro-tip: you can create a settings.xml file with these
>>>>>>>>>>>>> contents:
>>>>>>>>>>>>>
>>>>>>>>>>>>> <settings>
>>>>>>>>>>>>> <activeProfiles>
>>>>>>>>>>>>> <activeProfile>flink-1.4.0</activeProfile>
>>>>>>>>>>>>> </activeProfiles>
>>>>>>>>>>>>> <profiles>
>>>>>>>>>>>>> <profile>
>>>>>>>>>>>>> <id>flink-1.4.0</id>
>>>>>>>>>>>>> <repositories>
>>>>>>>>>>>>> <repository>
>>>>>>>>>>>>>   <id>flink-1.4.0</id>
>>>>>>>>>>>>>   <url>
>>>>>>>>>>>>> https://repository.apache.org/content/repositories/
>>>>>>>>>>>>> orgapacheflink-1141/
>>>>>>>>>>>>>   </url>
>>>>>>>>>>>>> </repository>
>>>>>>>>>>>>> <repository>
>>>>>>>>>>>>>   <id>archetype</id>
>>>>>>>>>>>>>   <url>
>>>>>>>>>>>>> https://repository.apache.org/content/repositories/
>>>>>>>>>>>>> orgapacheflink-1141/
>>>>>>>>>>>>>   </url>
>>>>>>>>>>>>> </repository>
>>>>>>>>>>>>> </repositories>
>>>>>>>>>>>>> </profile>
>>>>>>>>>>>>> </profiles>
>>>>>>>>>>>>> </settings>
>>>>>>>>>>>>>
>>>>>>>>>>>>> And reference that in you maven commands via --settings
>>>>>>>>>>>>> path/to/settings.xml. This is useful for creating a quickstart
>>>>>>>>>>>>>
>>>>>>>>>>>> based
>>>>
>>>>> on
>>>>>>>>>
>>>>>>>>>> the
>>>>>>>>>>>
>>>>>>>>>>>> staged release and for building against the staged jars.
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>
>>>>>>>
>>>>
>

Reply via email to