Adar Dembo has posted comments on this change. Change subject: Add basic Hive MetaStore client ......................................................................
Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/7053/5/src/kudu/hms/mini_hms.cc File src/kudu/hms/mini_hms.cc: Line 77: Env* env = Env::Default(); > Why is it better? It forces every caller to call Env::Default() instead. Not necessarily: when used from KuduTest, callers can pass env_ instead. http://gerrit.cloudera.org:8080/#/c/7053/5/thirdparty/download-thirdparty.sh File thirdparty/download-thirdparty.sh: Line 342: # Remove the share/doc dir which contains 2GiB of HTML project documentation. > We are forced to do that because LLVM doesn't a unified tarball with the co It's not an issue of forcing; we could consume unmodified LLVM tarballs and "reassemble" the source tree locally. How much space do the docs consume post-compression? -- To view, visit http://gerrit.cloudera.org:8080/7053 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I155223da912bc18a759df2f1f6bc25d1132a99ee Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
