Hi, +1 binding but there a few minor issues that need looking into
I checked: - incubating in release name - signatures and hashes good - DISCLAIMER exists - LICENSE may have a couple of minor issue (see below) - NOTICE good - No unexpected binary in release - A few minor header issues - Can compile from source These files [1][2][3][4] have "Copyright 2012 The Netty Project” rather than 2014 as mentioned in the NOTICE file. These files [5][6][7][8][9][10] have "Copyright 1999-2011 Alibaba Group” which I assume is not correct as they were part of the donation? I think this has been brought up previously. It looks like that the header for “ExecutionList” has been replaced with an incorrect one? It's mentioned in LICENSE. It also seem that the code from ListenableFuture is no longer in the release but still mentioned in LICENSE? Please fix this in the next release. If changes are made to any 3rd part software in the release be sure to update LICENSE and NOTICE. Thanks, Justin 1. ./dubbo-common/src/main/java/org/apache/dubbo/common/timer/HashedWheelTimer.java 2. ./dubbo-common/src/main/java/org/apache/dubbo/common/timer/Timeout.java 3. ./dubbo-common/src/main/java/org/apache/dubbo/common/timer/Timer.java 4. ./dubbo-common/src/main/java/org/apache/dubbo/common/timer/TimerTask.java 5. ./dubbo-rpc/dubbo-rpc-rest/pom.xml 6. ./dubbo-common/src/test/resources/log4j.xml 7. ./dubbo-test/dubbo-test-compatibility/pom.xml 8. ./dubbo-remoting/dubbo-remoting-netty/pom.xml 9. ./dubbo-remoting/dubbo-remoting-zookeeper/pom.xml 10. ./dubbo-monitor/dubbo-monitor-api/pom.xml --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
