+1 (non-binding)

Checked items:
- checked checksums and GPG files
- verified that the source archives do not contains any binaries
- checked that all POM files point to the same version
- build from source successfully

Best,
Kurt


On Tue, Mar 26, 2019 at 10:57 AM Shaoxuan Wang <wshaox...@gmail.com> wrote:

> +1 (non-binding)
>
> I tested RC4 with the following items:
> - Maven Central Repository contains all artifacts
> - Built the source with Maven (ensured all source files have Apache
> headers), and executed built-in tests via "mvn clean verify"
> - Manually executed the tests in IntelliJ IDE
> - Verify that the quickstarts for Scala and Java are working with the
> staging repository in IntelliJ
> - Checked the benchmark results. The perf regression of
> tuple-key-by/statebackend/tumblingWindow are gone, but the regression on
> serializer still exists.
>
> Regards,
> Shaoxuan
>
> On Tue, Mar 26, 2019 at 8:06 AM jincheng sun <sunjincheng...@gmail.com>
> wrote:
>
> > Hi Aljoscha, I think you are right, increase the timeout config will fix
> > this issue. this depends on the resource of Travis.  I would like share
> > some phenomenon during my test (not the flink problem) as follows:  :-)
> >
> > During my testing, `mvn clean verify` and `nightly end-to-end test ` both
> > consume a lot of machine resources (especially memory/network), and the
> > network bandwidth requirements of `nightly end-to-end test ` are also
> very
> > high. In China, need to use VPN acceleration (100~200Kb before
> > acceleration, 3~4Mb after acceleration), I have encountered: [Avro
> > Confluent Schema Registry nightly end-to-end test' failed after 18
> minutes
> > and 15 seconds! Test exited with exit Code 1] takes more than 18 minutes,
> > the download failed because the network bandwidth is not enough. and it
> > runs smoothly when using VPN acceleration. The overall end-to-end run was
> > passed twice. The Docker resource configuration (CUPs 7, Mem: 28.7G,
> Swap:
> > 3.5G). See detail log here
> > <
> >
> https://docs.google.com/document/d/1CcyTCyZmMmP57pkKv4drjSuxW61_u78HR3q1fJJODMw/edit?usp=sharing
> > >
> > .
> >
> > Just now, I had checked the Travis for your last commit (Increase startup
> > timeout in end-to-end tests), in addition to the Cleanup phase, other
> > phases are successful. here
> > <https://travis-ci.org/apache/flink/builds/511071777>
> >
> > In order to verify that our speculation is accurate, I can help with 10
> and
> > 20 seconds timeout config on my repo verification to see if 100%
> recurring
> > timeout problem. It is already running, we are waiting for the result.
> > 10seconds <https://travis-ci.org/sunjincheng121/flink/builds/511235749>
> > 20seconds <https://travis-ci.org/sunjincheng121/flink/builds/511235598>
> >
> > Best,
> > Jincheng
> >
> > Aljoscha Krettek <aljos...@apache.org> 于2019年3月26日周二 上午1:04写道:
> >
> > > Thanks for the testing done so far!
> > >
> > > There has been quite some flakiness on Travis lately, see here:
> > > https://travis-ci.org/apache/flink/branches <
> > > https://travis-ci.org/apache/flink/branches>. I’m a bit hesitant to
> > > release in this state. Looking at the tests you can see that all of the
> > > end-to-end tests fail because waiting for the dispatcher to come up
> times
> > > out. I also noticed that this usually takes about 5-8 seconds on
> Travis,
> > so
> > > a 10 second timeout might be a bit low. I pushed commits to increase
> that
> > > to 20 secs. Let’s see what will happen.
> > >
> > > I’ll keep you posted!
> > > Aljoscha
> > >
> > > > On 25. Mar 2019, at 13:13, jincheng sun <sunjincheng...@gmail.com>
> > > wrote:
> > > >
> > > > Great thanks for preparing the RC4 of Flink 1.8.0, Aljoscha!
> > > >
> > > > +1 (non-binding)
> > > >
> > > > I checked the functional things as follows(Without performance
> > > > verification):
> > > >
> > > > 1. Checking Artifacts:
> > > >
> > > >    1). Download the release source code -  SUCCESS
> > > >    2). Check Source release flink-1.8.0-src.tgz.sha512 - SUCCESS
> > > >    3). Download the released JAR - SUCCESS
> > > >    4). Check if checksums and GPG files match the corresponding
> release
> > > > files - SUCCESS.
> > > >    5). Verify that the source archives do not contain any binaries -
> > > > SUCCESS.
> > > >    6). Build the source with `mvn clean verify -DskipTests` to ensure
> > all
> > > > source files have Apache headers - SUCCESS
> > > >    7). Check that all POM files point to the same version - SUCCESS
> > > >    8). Read the `README.md` file to ensure there is nothing
> unexpected
> > -
> > > > SUCCESS
> > > >
> > > > 2. Testing Larger Setups
> > > >
> > > >   Cluster Environment:7 nodes, jm 1024m, tm 4096m
> > > >   Testing Jobs: WordCount(Batch&Streaming),
> > DataStreamAllroundTestProgram
> > > >
> > > >   1). Use local&hdfs file systems for checkpoints - SUCCESS
> > > >   2). Use hdfs file systems for input/output -SUCCESS
> > > >   3). Run examples on YARN(with or without session) - SUCCESS
> > > >   4). Test failover and recovery. - SUCCESS
> > > >   5). Test incremental&non-incremental checkpoint - SUCCESS
> > > >   6). Test connector - kafka -SUCCESS
> > > >
> > > > 3. Testing Functionality
> > > >
> > > >   1). Built-in tests(linux&mac os)
> > > >      - `mvn cealn verify`  (some test timeout error and test case bug
> > see
> > > > FLINK-12001 <https://issues.apache.org/jira/browse/FLINK-12001>, all
> > of
> > > > them are not the blocker)
> > > >      -  build for scala 2.11(mvn clean install -P scala-2.11
> > -DskipTests)
> > > > - SUCCESS
> > > >      -  Run the scripted nightly end-to-end test  - SUCCESS
> > > >
> > > >   2). Quickstarts
> > > >      - Verify that the quickstarts for Scala with the staging
> > repository
> > > > in IntelliJ - SUCCESS
> > > >      - Verify that the quickstarts for Java with the staging
> repository
> > > in
> > > > IntelliJ - SUCCESS
> > > >
> > > >   3). Simple Starter Experience and Use Cases
> > > >
> > > >      - run all examples from IntelliJ IDE  -  SUCCESS
> > > >      - Start a local cluster and verify that the processes -  SUCCESS
> > > >        a. Examine the *.out files (should be empty) and the log files
> > > > (should contain no exceptions)
> > > >        b. Test for Linux, MacOS
> > > >        c. Shutdown and verify there are no exceptions in the log
> output
> > > > (after shutdown)
> > > >
> > > >      - Verify that the examples are running from both ./bin/flink and
> > > from
> > > > the web-based job submission tool(following items)   -  SUCCESS
> > > >        a. Start multiple task managers in the local cluster
> > > >        b. Change the flink-conf.yml to define more than one task slot
> > (2)
> > > >        c. Run the examples with a parallelism > 1
> > > >        d. Examine the log output - no error messages should be
> > > encountered
> > > >
> > > > 4. Review the PR
> > > >     - [Add 1.8 Release Blog Post] - Just a reminder, updated the
> > release
> > > > date to correct date before merging.
> > > >
> > > > Cheers,
> > > > Jincheng
> > > >
> > > > Piotr Nowojski <pi...@ververica.com> 于2019年3月25日周一 下午4:11写道:
> > > >
> > > >> +1 from my side. Previously spotted performance regression seems to
> be
> > > >> gone, or mostly gone.
> > > >>
> > > >> Piotrek
> > > >>
> > > >>> On 21 Mar 2019, at 17:52, Aljoscha Krettek <aljos...@apache.org>
> > > wrote:
> > > >>>
> > > >>> Hi everyone,
> > > >>> Please review and vote on the release candidate 4 for Flink 1.8.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.8.0-rc4" [5],
> > > >>> * website pull request listing the new release [6]
> > > >>> * website pull request adding announcement blog post [7].
> > > >>>
> > > >>> The vote will be open for at least 72 hours. It is adopted by
> > majority
> > > >> approval, with at least 3 PMC affirmative votes.
> > > >>>
> > > >>> Thanks,
> > > >>> Aljoscha
> > > >>>
> > > >>> [1]
> > > >>
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12344274
> > > >>> [2] https://dist.apache.org/repos/dist/dev/flink/flink-1.8.0-rc4/
> > > >>> [3] https://dist.apache.org/repos/dist/release/flink/KEYS
> > > >>> [4]
> > > >>
> > https://repository.apache.org/content/repositories/orgapacheflink-1215
> > > >>> [5]
> > > >>
> > >
> >
> https://gitbox.apache.org/repos/asf?p=flink.git;a=tag;h=c650befc10c8bb6cc4b007ae250b7b2173046145
> > > >>> [6] https://github.com/apache/flink-web/pull/180 <
> > > >> https://github.com/apache/flink-web/pull/180>
> > > >>> [7] https://github.com/apache/flink-web/pull/179 <
> > > >> https://github.com/apache/flink-web/pull/179>
> > > >>>
> > > >>> P.S. The difference to the previous RCs is small, you can fetch the
> > > tags
> > > >> and do a "git log release-1.8.0-rc1..release-1.8.0-rc4” to see the
> > > >> difference in commits. Its fixes for the issues that led to the
> > > >> cancellation of the previous RCs plus smaller fixes. Most
> > > >> verification/testing that was carried out should apply as is to this
> > RC.
> > > >> Any functional verification that you did on previous RCs should
> > > therefore
> > > >> easily carry over to this one.
> > > >>
> > > >>
> > >
> > >
> >
>

Reply via email to