incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Justin Mclean <>
Subject Re: [VOTE] Apache Kudu (incubating) 0.7.0 RC3
Date Tue, 23 Feb 2016 00:32:45 GMT

+1 (binding). Nice work on the LICENSE.

I checked:
- signature and hashes correct
- release name contain incubating
- LICENSE file has some minor issues (see below)
- NOTICE good
- unable to compile on OS X (but notes say it only has experiment support)

Minor issues LICENSE file:
- BSD for Async HBase has "Google Inc.” in the 3rd clause probably a copy and paste error?
- Path to WebRTC licensed files should include src/kudu/util/random.h
- Missing license in LICENSE for FindGMock [1]
- Possible incorrect Apache header on BSD license in file [2]? Should also be in LICENSE.
- Header with copyright Cloudera which should be ASF? [3]
- Short form i.e. pointers to license file are preferred.

There is possibly a more serious issue with the licensing of this file [4]. See also [5][6].

From inside the file:
"Permission to make digital or hard copies of all or part of this work for personal or classroom
use is granted without fee provided that copies are not made or distributed for profit or
commercial advantage and that copies bear this notice and the full citation on the first page.
 To copy otherwise, to republish, to post on servers or to redistribute to lists, requires
prior specific permission and/or a fee.”

May be serious enough for another RC? IMO Up to the RM/PMC to decide that or fix in the next
incubating release.

A few other minor things:
- NOTICE file file line should probably be "Apache Kudu (incubating)" rather than "Apache
- There’s another github mirror here - does anyone else
think that a little odd?

JFYI The OSX compile error was (after about 1/2 hour of compiling things):
+ make -j8 install
CDPATH="${ZSH_VERSION+.}:" && cd . && aclocal-1.14 -I m4
/bin/sh: aclocal-1.14: command not found
make: *** [aclocal.m4] Error 127


1. ./apache-kudu-incubating-0.7.0/cmake_modules/FindGMock.cmake
2. ./apache-kudu-incubating-0.7.0/cmake_modules/FindProtobuf.cmake
3. ./apache-kudu-incubating-0.7.0/python/Makefile
4. ./apache-kudu-incubating-0.7.0/docs/whitepaper/vldb.cls

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message