Hi, There are several files missing license headers - is this a concern? ./dev/phoenix.importorder ./examples/STOCK_SYMBOL.csv ./examples/WEB_STAT.csv ./examples/pig/test.pig ./examples/pig/testdata ./phoenix-core/src/main/resources/java.sql.Driver
(ignoring CHANGES and rat.txt files) Some very minor issues that you may want to fix/change: - build.txt has "copyright 2010" in it - build.text should probably be called BUILDING - there's no .zip package - not require but convenient for users - source package name doesn't include "apache", not required you may want to consider adding this for trademark protection Thanks, Justin --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
