The artifacts should definitely not have rcX in their names. I mentioned this last time [1], but didn't notice it again this time (sorry about that!). I think the artifacts should be regenerated and the vote run again since we can't change the artifacts after the vote.
As Brock says, we should have a documented release procedure that each RM follows. Ryan would you like to create one on the wiki using the steps that you followed? Cheers, Tom [1] http://mail-archives.eu.apache.org/mod_mbox/incubator-parquet-dev/201409.mbox/%3ccabqr8u-rqj00si206alwherbf97rhtvcwvhrqwrqgy+ubh4...@mail.gmail.com%3E On Mon, Nov 17, 2014 at 7:12 PM, Brock Noland <[email protected]> wrote: > It looks like 1.6 was released with the rcX name inside so I am guessing > that procedure will work. > > I might revise the parquet release process in the future. The tarballs and > maven artifacts have the rc2 name inside the tarball since the name "rcX" > was in the version in the pom. I don't think that the rcX should be in the > pom version for the following reasons: > > 1) A release vote is on a specific source control tag. That tag should not > be updated post release thus artifacts will always have an rcX name. > 2) Having the rc number inside the pom results in tarballs with directories > which contain the rc name which is confusing when the tarball itself does > not. > > MRUnit can be used a reference on releasing from Apache. > > http://mrunit.apache.org/pmc/how_to_release.html > > Brock > > On Mon, Nov 17, 2014 at 10:55 AM, Ryan Blue <[email protected]> wrote: > > > Hi everyone, > > > > The parquet-format 2.2.0 release has been approved by the IPMC, so I'm > > going to do the release. I have access to create the content in > > /dist/release/incubator, but I want to confirm the next steps: > > > > 1. Copy all of the rc2 files from the dist/dev/incubator/parquet > directory > > to dist/dev/incubator/parquet/parquet-format-2.2.0 > > 2. Rename the files to remove "rc2" from the version string > > 3. Commit the changes, which actually releases the content > > 4. Delete the 2.2.0-rc* directories from the dist/dev area. > > > > Does that sound correct? > > > > rb > > > > > > On 11/14/2014 11:12 AM, Ryan Blue wrote: > > > >> This vote has passed with 3 +1 votes from IPMC members. > >> > >> Here are the votes: > >> > >> +1 (binding): > >> * Tom White (from podling vote) > >> * Stack > >> * Patrick Hunt > >> > >> +1 (non-binding): > >> * Chris Aniszczyk > >> * Julien Le Dem > >> > >> +0: (none) > >> -1: (none) > >> > >> Thank you to everyone for verifying the release! > >> > >> rb > >> > >> On 11/10/2014 02:35 PM, Ryan Blue wrote: > >> > >>> Hi everyone, > >>> > >>> I'd like to propose a vote to release parquet-format-2.2.0-rc2 as the > >>> official Parquet Format 2.2.0 release. > >>> > >>> This release candidate has passed a podling vote, which can be found > >>> here: > >>> > >>> https://mail-archives.apache.org/mod_mbox/incubator- > >>> parquet-dev/201411.mbox/%3C54613B48.6060602%40apache.org%3E > >>> > >>> > >>> > >>> The release candidate, signature, and checksums are available at: > >>> https://dist.apache.org/repos/dist/dev/incubator/parquet/2.2.0-rc2/ > >>> > >>> The branch used to create the release candidate is: > >>> > >>> https://git-wip-us.apache.org/repos/asf?p=incubator-parquet- > >>> format.git;hb=release-2.2.0-rc2 > >>> > >>> > >>> > >>> KEYS to validate the signature are available at: > >>> https://dist.apache.org/repos/dist/dev/incubator/parquet/KEYS > >>> > >>> Please download, verify, and test. > >>> > >>> [ ] +1: Release this tag as parquet-format-2.2.0 > >>> [ ] +0: > >>> [ ] -1: This release is not ready because . . . > >>> > >> > >> > >> > > > > -- > > Ryan Blue > > Software Engineer > > Cloudera, Inc. > > >
