+1 Regards Liang
Gian Merlino <[email protected]> 于2019年8月15日周四 上午12:04写道： > +1 (non-binding) > > Good stuff: > > - .asc and .sha512 look good. > - git.properties looks good. > - Ran "mvn package" without issue. > - Source tarball contents match the tag. > > Things to consider for future releases: > > - I don't think this is a requirement, but it's more normal for the release > to unpack to a directory named > "apache-datasketches-memory-1.1.0-incubating-src" rather than the current > directory. That way, users can simply download and unzip the archive, > without needing to create a create an empty directory to hold it first. I > think achieving this is a matter of editing your assembly xml. > - The DISCLAIMER file links to > http://incubator.apache.org/project/#Podling-Name#.html, but it should be > https://incubator.apache.org/projects/datasketches.html. (fill in project > name, 'projects' instead of 'project', http would work but https is nicer.) > I raised this at https://github.com/apache/incubator/pull/29 too. > - The DISCLAIMER file should be named DISCLAIMER-WIP per > https://incubator.apache.org/policy/incubation.html#disclaimers. > - LICENSE has the Apache license as expected. I didn't look too closely at > the rest since the disclaimer said it's still a work in progress. > - NOTICE contains the text "Prior to moving to ASF, the software for this > project was developed at Yahoo (now Verizon Media) ( > https://developer.yahoo.com)." Some musings: NOTICE isn't supposed to > contain anything that is not legally required, per > http://www.apache.org/dev/licensing-howto.html. Do you know if this > statement is legally required? If so, include it. If not, honestly I'm not > totally sure what the right thing is to do is. Some Apache projects sourced > from corporate donations include lines like this in their NOTICE files, and > some don't. I think if I was you I would either remove it, or add it > underneath a copyright line of the same style as was in the original source > files, assuming you removed them and replaced them with ASF headers > (because NOTICE can be a home for relocated copyright lines, per > http://www.apache.org/dev/licensing-howto.html#mod-notice). Fwiw, > assembling a proper NOTICE file has been a struggle for the Druid project, > and I feel we've got some murky and conflicting advice over the past year. > So please take my advice here with a grain of salt. > - Vote procedure stuff: you could also mention that a release requires not > just three positive votes, but also a majority (more +1 than -1) per > https://www.apache.org/foundation/voting.html. This doesn't usually come > up > though (from what I've seen, usually if someone votes -1 the release > manager will either cancel the vote or convince that person to flip to +1). > > On Tue, Aug 13, 2019 at 5:21 PM leerho <[email protected]> wrote: > > > Hello Apache DataSketches PPMC and Community, > > > > 1. This is a call for vote to release Apache DataSketches-memory version: > > 1.1.0-incubating-RC2 > > > > NOTE 1: This is one component of the DataSketches library that provides > > high-performance access > > to off-heap memory for Java applications. These are low-level functions > > that are used by other > > DataSketches components. It does not provide any command-line interface > or > > GUI. > > > > NOTE 2: This release contains some critical performance improvements for > > Apache Druid. > > > > NOTE 3: This particular component has been split out from the rest of the > > DataSketches > > code because there have been a number of requests from platform projects > > that are interested in > > these high-performance functions independently of the remainder of the > > DataSketches library. > > > > NOTE 4: This RC2 fixes some problems in the structure of the POM file of > > RC1. There are no other changes. > > > > 2. The release candidate: > > > > > https://dist.apache.org/repos/dist/dev/incubator/datasketches/memory/1.1.0-incubating-RC2/ > > > > 3. Source repository: > > https://github.com/apache/incubator-datasketches-memory > > > > Git Tag for this release: > > > > > https://github.com/apache/incubator-datasketches-memory/tree/1.1.0-incubating-RC2 > > on branch 1.1.X-incubating > > > > Git HashId for this release starts with: d94fcef > > > > The artifacts have been signed with --keyid-format SHORT : 8CD4A902 > > > > 4. Repository: dist.apache.org repository: > > > > The public signing key can be found in the KEYS file: > > https://dist.apache.org/repos/dist/dev/incubator/datasketches/KEYS > > > > 5. Repository: Maven Central (repository.apache.org): > > > > The Jar artifacts can be found at > > > > > https://repository.apache.org/content/groups/staging/org/apache/datasketches/datasketches-memory/1.1.0-incubating/ > > > > 6. Build & Test Guide: > > > > The DataSketches-memory component is pure Java and is structured as a > Maven > > project. You must compile using JDK 8. > > There is one run-time dependency for logging and, of course, a number of > > test and Maven plugin dependencies, > > all of which can be resolved by Maven. > > > > There are two types of tests: normal unit tests and tests run by the > strict > > profile. To run normal unit tests: > > > > $ mvn clean test > > > > To run the strict profile tests: > > > > $ mvn clean test -P strict > > > > To install jars in your local .m2 repository: > > > > $ mvn clean install > > > > 7. Documentation: The documentation for the DataSketches Memory component > > is part of the website. > > > > Overview documentation: > > - https://datasketches.github.io/docs/Memory/MemoryPackage.html > > - https://datasketches.github.io/docs/Memory/MemoryPerformance.html > > > > Javadocs: > > https://datasketches.github.io/api/memory/snapshot/apidocs/index.html > > > > > > 8. The vote will be performed in two stages: > > - This letter will be published on dev@ and remain open for at least > 72 > > hours and at least 3 (+1) PPMC votes are reached. > > All PPMC members including Mentors can vote. However, a negative > vode > > from a Mentor will cancel this voting process. > > The 72 hours will elapse as of Friday, August 16, 2019, PDT (UTC -7 > > hours). > > > > - After it passes the first stage, the summary of that vote and the > key > > information from this letter will be > > published on general@incubator and remain open for at least 72 > hours > > and at least 3 (+1) IPMC votes are reached. > > > > Please vote accordingly: > > > > [ ] +1 approve > > [ ] +0 no opinion > > [ ] -1 disapprove with the reason > > > > Thanks, > > Lee Rhodes > > [email protected] > > > -- Regards Liang
