+1, I have checked the following:

- Built both 2.10 and 2.11 from source (skipped tests)
- Checksums and signatures
- NOTICE and LICENSE
- Rat

Questions:
1- I'm wondering if the text about Hadoop in NOTICE is necessary. How did you 
guys end up including it?
2- The tgz bundles do not include any jar directly, so there is no real concern 
about bundling the bits from other projects that could require more sections in 
the NOTICE file, is it right?

-Flavio

> On 17 Jan 2017, at 17:12, Leigh Stewart <lstew...@twitter.com.INVALID> wrote:
> 
> +1
> 
> On Mon, Jan 16, 2017 at 8:13 AM, Jon Derrick <jonathan.derri...@gmail.com>
> wrote:
> 
>> +1
>> 
>> LGTM. compiled the source packages and ran dbench. the license files look
>> good.
>> 
>> - jd
>> 
>> On Tue, Jan 10, 2017 at 11:56 PM, Sijie Guo <si...@apache.org> wrote:
>> 
>>> Hi all,
>>> 
>>> Please review and vote on the release candidate #2 for the version 0.4.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 to be deployed to
>> dist.apache.org
>>> [2],
>>>    * all artifacts to be deployed to the Maven Central Repository
>> [3][4],
>>>    * source code tag "v0.4.0-incubating-RC1_2.11" (for scala 2.11) and
>>> "v0.4.0-incubating-RC1_2.10" (for scala 2.10) [5][6],
>>>    * website pull request listing the release [7] and publishing the API
>>> reference manual.
>>> 
>>> A simple instruction for validation the source and binary packages.
>>> 
>>> - source package: building the package with "*mvn clean apache-rat:check
>>> package findbugs:check -DskipTests*"
>>> 
>>> The vote will be open for at least 72 hours. It is adopted by majority
>>> approval, with at least 3 PPMC affirmative votes.
>>> 
>>> Thanks,
>>> Sijie
>>> 
>>> [1] https://issues.apache.org/jira/secure/ReleaseNote.jspa?
>>> projectId=12320620&version=12337980
>>> [2]
>>> https://dist.apache.org/repos/dist/dev/incubator/distributedlog/0.4.0-
>>> incubating-RC2/
>>> [3]
>>> https://repository.apache.org/content/repositories/
>>> orgapachedistributedlog-1003/
>>> [4]
>>> https://repository.apache.org/content/repositories/
>>> orgapachedistributedlog-1004/
>>> [5]
>>> https://github.com/apache/incubator-distributedlog/tree/
>>> v0.4.0-incubating-RC1_2.11
>>> [6]
>>> https://github.com/apache/incubator-distributedlog/tree/
>>> v0.4.0-incubating-RC1_2.10
>>> [7] https://github.com/apache/incubator-distributedlog/pull/109
>>> 
>> 
>> 
>> 
>> --
>> - jderrick
>> 

Reply via email to