[kudu-CR] Expose running maintenance op info

2017-08-02 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#5). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Add Maintenance Manager visualizer

2017-08-02 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7570 Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager visualizer Adds a timeline/swimlane chart to display

[kudu-CR] Expose running maintenance op info

2017-08-01 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#4). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-08-01 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 3: (9 comments) http://gerrit.cloudera.org:8080/#/c/7537/3/src/kudu/tserver/tserver-path-handlers.cc File

[kudu-CR] Expose running maintenance op info

2017-08-01 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#3). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-08-01 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 2: (11 comments) http://gerrit.cloudera.org:8080/#/c/7537/2/src/kudu/tserver/tserver-path-handlers.h File

[kudu-CR] Add Maintenance Manager visualizer

2017-08-03 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add Maintenance Manager visualizer .. Patch Set 1: The last two builds failed with different tests, and both seem unrelated. -- To view, visit

[kudu-CR] Add json builder utility

2017-07-11 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/7387/3/src/kudu/util/easy_json.h File src/kudu/util/easy_json.h: PS3,

[kudu-CR] Add json builder utility

2017-07-11 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/7387/3/src/kudu/util/easy_json.h File src/kudu/util/easy_json.h: Line

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7387/1//COMMIT_MSG Commit Message: PS1, Line 10: It is intended solely

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7387/1/src/kudu/util/jsonbuilder.h File src/kudu/util/jsonbuilder.h:

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7387 to look at the new patch set (#2). Change subject: Add json builder utility .. Add json builder utility The

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7387 Change subject: Add json builder utility .. Add json builder utility The EasyJson class serves as a wrapper around rapidjson Value

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7387 to look at the new patch set (#3). Change subject: Add json builder utility .. Add json builder utility The

[kudu-CR] Add json builder utility

2017-07-10 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7387/1/src/kudu/util/jsonbuilder.h File src/kudu/util/jsonbuilder.h:

[kudu-CR] Add json builder utility

2017-07-12 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add json builder utility .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7387/4/src/kudu/util/CMakeLists.txt File src/kudu/util/CMakeLists.txt:

[kudu-CR] Add json builder utility

2017-07-12 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7387 to look at the new patch set (#5). Change subject: Add json builder utility .. Add json builder utility The

[kudu-CR] Add json builder utility

2017-07-12 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7387 to look at the new patch set (#6). Change subject: Add json builder utility .. Add json builder

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#4). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#5). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7448 Change subject: Integrate Mustache templates to webserver .. Integrate Mustache templates to webserver This commit alters the design of

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#2). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-17 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#3). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#6). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Add security dashboard to web UI

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7505 Change subject: Add security dashboard to web UI .. Add security dashboard to web UI Adds a new "Configuration" page to the web UI that

[kudu-CR] Switch tables page to template

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7506 Change subject: Switch tables page to template .. Switch tables page to template Converts the /tables page of the master web UI to use a

[kudu-CR] Update web UI style

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7504 Change subject: Update web UI style .. Update web UI style Update to latest versions of jquery and bootstrap, and make some small

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#10). Change subject: Integrate Mustache templates to webserver .. Integrate

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#11). Change subject: Integrate Mustache templates to webserver .. Integrate

[kudu-CR] Integrate Mustache templates to webserver

2017-07-24 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#12). Change subject: Integrate Mustache templates to webserver .. Integrate

[kudu-CR] Integrate Mustache templates to webserver

2017-07-26 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#14). Change subject: Integrate Mustache templates to webserver .. Integrate

[kudu-CR] Update web UI style

2017-07-26 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7504 to look at the new patch set (#3). Change subject: Update web UI style .. Update web UI style Update

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#9). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#8). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#7). Change subject: Integrate Mustache templates to webserver .. Integrate Mustache

[kudu-CR] Integrate Mustache templates to webserver

2017-07-19 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 9: This commit didn't go anywhere near raft consensus, and the log doesn't look like the failure of

[kudu-CR] Add json builder utility

2017-07-11 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7387 to look at the new patch set (#4). Change subject: Add json builder utility .. Add json builder utility The

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#7). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#8). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#6). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/7537/5/src/kudu/util/maintenance_manager.cc File

[kudu-CR] Expose running maintenance op info

2017-08-04 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Expose running maintenance op info .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/7537/7/src/kudu/tserver/tserver-path-handlers.cc File

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7570 to look at the new patch set (#5). Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager

[kudu-CR] Add Maintenance Manager visualizer

2017-08-04 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7570 to look at the new patch set (#6). Change subject: Add Maintenance Manager visualizer .. Add Maintenance Manager

[kudu-CR] Test Prepare() failed codepath

2017-08-04 Thread Sam Okrent (Code Review)
Sam Okrent has abandoned this change. Change subject: Test Prepare() failed codepath .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7522 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[kudu-CR] Expose running maintenance op info

2017-07-28 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7537 Change subject: Expose running maintenance op info .. Expose running maintenance op info Previously, the maintenance manager stored the

[kudu-CR] Expose running maintenance op info

2017-07-28 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7537 to look at the new patch set (#2). Change subject: Expose running maintenance op info .. Expose running

[kudu-CR] Update web UI style

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Update web UI style .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/7504/1/www/bootstrap/js/bootstrap.js File www/bootstrap/js/bootstrap.js:

[kudu-CR] Add security dashboard to web UI

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add security dashboard to web UI .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7505/1/src/kudu/server/default-path-handlers.cc File

[kudu-CR] Update web UI style

2017-07-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7504 to look at the new patch set (#2). Change subject: Update web UI style .. Update web UI style Update to latest

[kudu-CR] Add security dashboard to web UI

2017-07-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7505 to look at the new patch set (#2). Change subject: Add security dashboard to web UI .. Add security dashboard to

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7448 to look at the new patch set (#13). Change subject: Integrate Mustache templates to webserver .. Integrate

[kudu-CR] Integrate Mustache templates to webserver

2017-07-25 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Integrate Mustache templates to webserver .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/7448/12/src/kudu/server/webserver.cc File

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-09 Thread Sam Okrent (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7111 to look at the new patch set (#5). Change subject: KUDU-1442 log number of open log block containers ..

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-09 Thread Sam Okrent (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7111 to look at the new patch set (#6). Change subject: KUDU-1442 log number of open log block containers ..

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-21 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1955 refuse to use world-readable keytabs .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/7249/2/src/kudu/rpc/messenger.cc File

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-21 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7249 Change subject: KUDU-1955 refuse to use world-readable keytabs .. KUDU-1955 refuse to use world-readable keytabs Allowing users to supply

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-22 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7249 to look at the new patch set (#3). Change subject: KUDU-1955 refuse to use world-readable keytabs .. KUDU-1955

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-22 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7249 to look at the new patch set (#4). Change subject: KUDU-1955 refuse to use world-readable keytabs .. KUDU-1955

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-22 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7249 to look at the new patch set (#5). Change subject: KUDU-1955 refuse to use world-readable keytabs .. KUDU-1955

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-22 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1955 refuse to use world-readable keytabs .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7249/2/src/kudu/util/env.h File

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-22 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1955 refuse to use world-readable keytabs .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-23 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1955 refuse to use world-readable keytabs .. Patch Set 5: (4 comments)

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-23 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1955 refuse to use world-readable keytabs .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/7249/6/src/kudu/security/init.cc File

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-23 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7249 to look at the new patch set (#6). Change subject: KUDU-1955 refuse to use world-readable keytabs ..

[kudu-CR] KUDU-1955 refuse to use world-readable keytabs

2017-06-23 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7249 to look at the new patch set (#7). Change subject: KUDU-1955 refuse to use world-readable keytabs

[kudu-CR] wip

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7297 Change subject: wip .. wip Change-Id: I1035f1faa5a95a44dc70793daf26ef68f689c66b --- M thirdparty/vars.sh 1 file changed, 12 insertions(+),

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7298 Change subject: Add mustache as a dependency .. Add mustache as a dependency Mustache is a popular template engine. The version added by

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new patch set (#2). Change subject: Add mustache as a dependency .. Add mustache as a dependency Mustache is a popular template engine. The version added by this commit is a C++ implementation found at

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add mustache as a dependency .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7298/2/src/kudu/server/default-path-handlers.cc File

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add mustache as a dependency .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7298/2/thirdparty/build-definitions.sh File

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: Add mustache as a dependency .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/7298/2//COMMIT_MSG Commit Message: Line 7: Add mustache as a

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7298 to look at the new patch set (#3). Change subject: Add mustache as a dependency .. Add mustache as a dependency

[kudu-CR] Add mustache as a dependency

2017-06-26 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7298 to look at the new patch set (#4). Change subject: Add mustache as a dependency .. Add mustache as a

[kudu-CR] wip

2017-06-26 Thread Sam Okrent (Code Review)
Sam Okrent has abandoned this change. Change subject: wip .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7297 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: abandon

[kudu-CR] KUDU-1911 add more helpful error message when required tool arguments are missing

2017-05-24 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/6986 Change subject: KUDU-1911 add more helpful error message when required tool arguments are missing .. KUDU-1911 add more helpful error

[kudu-CR] KUDU-1911 improve missing required arg message

2017-05-26 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6986 to look at the new patch set (#5). Change subject: KUDU-1911 improve missing required arg message ..

[kudu-CR] KUDU-1911 improve missing required arg message

2017-05-26 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6986 to look at the new patch set (#6). Change subject: KUDU-1911 improve missing required arg message ..

[kudu-CR] KUDU-1911 improve missing required arg message

2017-05-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6986 to look at the new patch set (#2). Change subject: KUDU-1911 improve missing required arg message .. KUDU-1911

[kudu-CR] KUDU-1911 improve missing required arg message

2017-05-25 Thread Sam Okrent (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6986 to look at the new patch set (#4). Change subject: KUDU-1911 improve missing required arg message ..

[kudu-CR] KUDU-1911 improve missing required arg message

2017-05-25 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6986 to look at the new patch set (#3). Change subject: KUDU-1911 improve missing required arg message .. KUDU-1911

[kudu-CR] KUDU-1914 Add positive .htpasswd test case

2017-05-30 Thread Sam Okrent (Code Review)
Sam Okrent has uploaded a new change for review. http://gerrit.cloudera.org:8080/7026 Change subject: KUDU-1914 Add positive .htpasswd test case .. KUDU-1914 Add positive .htpasswd test case This commit adds a test to check

[kudu-CR] KUDU-1914 Add positive .htpasswd test case

2017-05-31 Thread Sam Okrent (Code Review)
Hello Hao Hao, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7026 to look at the new patch set (#2). Change subject: KUDU-1914 Add positive .htpasswd test case

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-08 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7111 to look at the new patch set (#2). Change subject: KUDU-1442 log number of open log block containers .. KUDU-1442

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-08 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1442 log number of open log block containers .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7111/1/src/kudu/fs/log_block_manager.cc File

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-09 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7111 to look at the new patch set (#3). Change subject: KUDU-1442 log number of open log block containers .. KUDU-1442

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-09 Thread Sam Okrent (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7111 to look at the new patch set (#4). Change subject: KUDU-1442 log number of open log block containers .. KUDU-1442

[kudu-CR] KUDU-1442 log number of open log block containers

2017-06-09 Thread Sam Okrent (Code Review)
Sam Okrent has posted comments on this change. Change subject: KUDU-1442 log number of open log block containers .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7111/3//COMMIT_MSG Commit Message: Line 20: