Hi,

-1 (binding) as incubating is not in the name. I suggest you rename and carry 
on the vote. If you do that treat my vote as +1 (binding).

I checked:
- incubating in not in name
- signatures and hashes are fine
- DISCLAIMER exists (WIP)
- LICENSE and NOTICE need more work
- didn’t compile from source.

I did also note CDDL licensed code in the release that we needs to be 
removed.This issue and others that have brought up in previous releases also 
haven’t been fixed, when is the PPMC intending to fix these?

I number of files seem to have confused licensed e.g this file has 3 license 
headers which is the correct license? [1]

Thanks,
Justin

1.  ./src/brpc/ts.cpp
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to