Thank you for your detailed review! We will address your comments on signing keys in the future votes.
Regarding the commit id, "44c9323" is the correct commit id for the tag. Maybe the link is not valid. Is https://github.com/apache/incubator-druid/commits/44c9323 or https://github.com/apache/incubator-druid/tree/druid-0.15.0-incubating-rc2 more valid one? Finally, would you elaborate more on what looks wrong to you in NOTICE and NOTICE.BINARY files? We will fix them if necessary for this release. Jihoon On Thu, Jun 20, 2019 at 9:46 AM sebb <seb...@gmail.com> wrote: > On Thu, 20 Jun 2019 at 03:39, Jihoon Son <jihoon...@apache.org> wrote: > > > > Hi IPMC, > > > > The Apache Druid community has voted on and approved a proposal to > release > > Apache Druid (incubating) 0.15.0 (rc2). > > > > We now kindly request the Incubator PMC members review and vote on this > > incubator release. > > > > Apache Druid (incubating) is a high performance analytics data store for > > event-driven data. > > > > The community voting thread can be found here: > > > https://lists.apache.org/thread.html/f4b1b708bcb7e6ec08e6a9cfcb2c0dfcb565fab353ccbb8c5f362218@%3Cdev.druid.apache.org%3E > > > > The release notes are available here: > > https://github.com/apache/incubator-druid/issues/7854 > > > > The release candidate has been tagged in GitHub as > > druid-0.15.0-incubating-rc2 (44c9323), available here: > > > https://github.com/apache/incubator-druid/releases/tag/druid-0.15.0-incubating-rc2 > > I'm not sure that's the correct URL for the tag; it points to a couple > of archives. > > I would expect a pointer to the source code. > This should use the commit id that corresponds to the release > candidate tag, i.e. the commit id that corresponds to > https://github.com/apache/incubator-druid/tree/druid-0.15.0-incubating-rc2 > AIUI only commit ids are truly immutable > > The NOTICE and NOTICE.BINARY files look wrong to me; they have a lot > of superfluous text. > > > The artifacts to be voted on are located here: > > > https://dist.apache.org/repos/dist/dev/incubator/druid/0.15.0-incubating-rc2/ > > > > A staged Maven repository is available for review at: > > https://repository.apache.org/content/repositories/orgapachedruid-1007/ > > > > Release artifacts are signed with the key [95574000]: > > https://people.apache.org/keys/committer/jihoonson.asc > > > > This key and the key of other committers can also be found in the > project's > > KEYS file here: > > https://dist.apache.org/repos/dist/release/incubator/druid/KEYS > > > > As part of the validation process, the release artifacts can be generated > > from source by running: > > mvn clean install -Papache-release,dist > > > > The RAT license check can be run from source by: > > mvn apache-rat:check -Prat > > > > This vote will be open for at least 72 hours. The vote will pass if a > > majority of at least three +1 IPMC votes are cast. > > > > [ ] +1 Release this package as Apache Druid (incubating) 0.15.0 > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > [ ] -1 Do not release this package because... > > > > Thank you IPMC! We appreciate your efforts in helping the Apache Druid > > community to validate this release. > > > > On behalf of the Apache Druid PPMC, > > Jihoon > > > > Apache Druid (incubating) is an effort undergoing incubation at The > Apache > > Software Foundation (ASF), sponsored by the Apache Incubator. Incubation > is > > required of all newly accepted projects until a further review indicates > > that the infrastructure, communications, and decision making process have > > stabilized in a manner consistent with other successful ASF projects. > While > > incubation status is not necessarily a reflection of the completeness or > > stability of the code, it does indicate that the project has yet to be > > fully endorsed by the ASF. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
