Hi,

+1 binding

I checked:
- file name contains incubating
- signature and hash good
- contains DISCLAIMER
- LICENSE and NOTICE good
- All source files have Apache headers
- No unexpected binary files in source release
- Can compile from source (on OS X it takes a while)

Only possible (very minor) thing I found is I think this file should be ASF 
licensed not Cloudera licensed? [1]

Also amusing to note that this file [2] has 13 Apache headers in it all 
copyright Twitter, no other files has duplicate headers that I could find.

The build instructions mentioned in the readme are hosted on a non Apache web 
site. In fact all URLs to the expected incubator site redirect to this external 
site. I assume at some point this will be moved to Apache infrastructure?

Thanks,
Justin

1. ./src/kudu/util/memory/overwrite.h
2. ./www/bootstrap/js/bootstrap.js
3. http://getkudu.io/docs/installation.html#_build_from_source


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to