WFM. I'll fix this JAR thing (https://issues.cloudera.org/browse/IMPALA-4171) then roll another RC. That one I'll take a vote on, and I'll change the wiki page with release instructions to say that the RM can jump into a vote after his or her own testing and can always roll another RC and take another vote even if one RC has passed or is about to pass.
Everyone else: please continue testing and verifying so we can fix as much stuff in RC3 as possible; thank you! Thanks, Tom and Brock! On Tue, Sep 20, 2016 at 12:44 PM, Brock Noland <br...@apache.org> wrote: > In this case, as RM, I just role another RC. Done that many times myself. > > AFAIK there isn't anything that "forces" you to release and that seems > contrary to everything I've been involved with at the ASF. > > On Tue, Sep 20, 2016 at 12:30 PM, Jim Apple <jbap...@cloudera.com> wrote: > > > I wanted to leave in our release procedures the opportunity for the > release > > manager to NOT release if there are three +1s but the RM finds something > he > > or she wants to fix first. For instance, what if we had had three +1s > > already before you found that JAR? Would the RM have to release? > > > > I don't know, so I added in the extra step. > > > > On Tue, Sep 20, 2016 at 10:08 AM, Tom White <t...@cloudera.com> wrote: > > > > > Any reason this isn't a vote? Usually an RC will be voted on, and if > > > problems are found then another RC will be created, until the PPMC is > > > happy with it, at which point it can go to the IPMC. > > > > > > I ran through the legal checks and it looks good to me. I noticed that > > > there's a binary file (testdata/udfs/impala-hive-udfs.jar), which > > > should really be removed. > > > > > > Cheers, > > > Tom > > > > > > > > > On Sun, Sep 18, 2016 at 1:15 AM, Jim Apple <jbap...@cloudera.com> > wrote: > > > > This is the second release candidate for Apache Impala (incubating) > > > 2.7.0: > > > > > > > > https://dist.apache.org/repos/dist/dev/incubator/impala/2.7.0/RC2/ > > > > > > > > The purpose of this thread is to gather feedback on the release > > > > candidate to see if a third release candidate is needed or if we can > > > > go ahead and vote on rc2. > > > > > > > > The git tag of the tree I made the tarball from is "2.7.0-rc2": > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > git;a=tag;h= > > > cabaa58fa5c15b0457c520a658dc74b4d174b1c7 > > > > > > > > That is commit: > > > > > > > > 53439d3d8cdca2dd1ca2cdf2c36c6a7ef101f7e3 > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=53439d3d8cdca2dd1ca2cdf2c36c6a7ef101f7e3 > > > > > > > > and this tree: > > > > > > > > 0b8e8a3f48b8086d53ee96064668dfccd56d57cc > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > git;a=tree;h= > > > 0b8e8a3f48b8086d53ee96064668dfccd56d57cc;hb= > > 53439d3d8cdca2dd1ca2cdf2c36c6a > > > 7ef101f7e3 > > > > > > > > ------------------------------------------------------------------ > > > > > > > > You can find instructions for how to evaluate a release candidate > here: > > > > > > > > https://cwiki.apache.org/confluence/display/IMPALA/ > > > DRAFT%3A+How+to+Release#DRAFT:HowtoRelease-HowtoVoteonaRelea > seCandidate > > > > > > > > Here is a copy of that section of the wiki: > > > > > > > > 1. Download the tarball. Check the signature and the checksums. > > > > > > > > # change to a new directory > > > > cd $(mktemp -d) > > > > > > > > # Download the keys of the release managers > > > > wget https://dist.apache.org/repos/dist/dev/incubator/impala/KEYS > > > > gpg --import KEYS > > > > > > > > # Set the keys of the release managers as trusted > > > > gpg --edit-key jbapple trust > > > > # At the prompt, enter '5' for "I trust ultimately", then 'y' for > > > > "yes", then 'q' for "quit" > > > > > > > > # Download the release artifacts: > > > > wget https://dist.apache.org/repos/dist/dev/incubator/impala/x.y. > > > z/RCq/apache-impala-incubating-x.y.z.tar.{gz,gz.asc,gz.md5,gz.sha} > > > > > > > > # Check the checksums: > > > > md5sum --check apache-impala-incubating-x.y.z.tar.gz.md5 > > > > sha1sum --check apache-impala-incubating-x.y.z.tar.gz.sha > > > > > > > > # Check the signature: > > > > gpg --verify apache-impala-incubating-x.y.z.tar.gz.asc > > > > > > > > 2. Check that it matches the upstream tag > > > > # move to your git directory and checkout the tag: > > > > cd incubator-impala > > > > git fetch apache --tags > > > > git checkout x.y.z-rcw > > > > > > > > # compare the tarball and the repo: > > > > cd .. > > > > tar xzf apache-impala-incubating-x.y.z.tar.gz > > > > diff -r apache-impala-incubating-x.y.z incubator-impala > > > > # You should see something like "Only in incubator-impala: .git", but > > > > no other output > > > > > > > > 3. Test the release quality, possibly using bin/run-all-tests.py. > > > > > > > > 4. Check compliance with ASF release policy. Use Apache RAT and > follow > > > > the instructions in bin/check-rat-report.py to check licence > > > > compliance. > > > > > > > > ------------------------------------------------------------ > > > ---------------------- > > > > > > > > Changes since the last release candidate: > > > > > > > > 1. Minor changes in the way the release candidate was made (directory > > > > name in tarball, location on dist.apache.org, filename). The current > > > > procedure is documented on > > > > https://cwiki.apache.org/confluence/pages/viewpage. > > > action?pageId=65869538 > > > > > > > > 2. Un-break building from non-git directories: > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=9f08d1ab3c876dc1fc92c9decb8104400eaeec7c > > > > > > > > 3. RAT script to check licensing: > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=85a0e36423c6c938c1a540629c16d27b3a9e1522 > > > > > > > > 4. License fixes: > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=586ae2d7f357d1b2fd0d7b59dad0229e7383747d > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=d385ac72d1400a6c1eb5da6de6f8c28aba33a3d4 > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=fb646d1b06a21f44518616bd49a91b88998ff602 > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=53439d3d8cdca2dd1ca2cdf2c36c6a7ef101f7e3 > > > > > > > > 5. Remove 'cdh' from version string: > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-impala. > > > git;a=commit;h=046267b72e5b63b134a4ef004ccde2a268a9ae42 > > > > > >