[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 10: (1 comment)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9373 to look at the new patch set (#11). Change subject: KUDU-2319 follower masters should be able to verify authn tokens

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 16: (5 comments)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#16). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR](branch-1.6.x) KUDU-1613: evict replicas with wrong server UUIDs

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9410 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. KUDU-1613: evict replicas with wrong server UUIDs

[kudu-CR](branch-1.6.x) util: move ListFilesInDir to env util

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9437 ) Change subject: util: move ListFilesInDir to env_util .. util: move ListFilesInDir to env_util ListFilesInDir() is useful

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9440 to look at the new patch set (#3). Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 15: (4 comments)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hello Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#15). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9440 ) Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic .. Patch Set 2: (1 comment)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9172 ) Change subject: maintenance_manager: log the reason for scheduling each operation .. Patch Set 1: OK, I made things a bit more concise,

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-02-23 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9172 to look at the new patch set (#2). Change subject: maintenance_manager: log the reason for scheduling each operation

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9440 ) Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic .. Patch Set 1: (4 comments)

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9440 to look at the new patch set (#2). Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9440 ) Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9440 ) Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic .. Patch Set 1: (3 comments)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9373 to look at the new patch set (#10). Change subject: KUDU-2319 follower masters should be able to verify authn tokens

[kudu-CR] KUDU-2312: Scan predicate application ordering is non-deterministic

2018-02-23 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9440 to review the following change. Change subject: KUDU-2312: Scan predicate application ordering is non-deterministic

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 14: (1 comment)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hello Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#14). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR](branch-1.6.x) KUDU-1613: evict replicas with wrong server UUIDs

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9410 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR](branch-1.6.x) util: move ListFilesInDir to env util

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9437 ) Change subject: util: move ListFilesInDir to env_util .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9373 to look at the new patch set (#9). Change subject: KUDU-2319 follower masters should be able to verify authn tokens

[kudu-CR] KUDU-2290: Tool to re-create a tablet

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9393 ) Change subject: KUDU-2290: Tool to re-create a tablet .. Patch Set 2: (4 comments)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 13: (1 comment)

[kudu-CR](branch-1.6.x) KUDU-1613: evict replicas with wrong server UUIDs

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9410 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 1: (1 comment)

[kudu-CR](branch-1.6.x) KUDU-1613: evict replicas with wrong server UUIDs

2018-02-23 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9410 to look at the new patch set (#2). Change subject: KUDU-1613: evict replicas with wrong server UUIDs

[kudu-CR](branch-1.6.x) util: move ListFilesInDir to env util

2018-02-23 Thread Andrew Wong (Code Review)
Hello David Ribeiro Alves, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9437 to review the following change. Change subject: util: move ListFilesInDir to env_util .. util:

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 6: (1 comment)

[kudu-CR] Don't perform compactions when clean time has not been advanced

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9436 ) Change subject: Don't perform compactions when clean time has not been advanced .. Don't perform compactions when clean

[kudu-CR] KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9329 ) Change subject: KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks .. KUDU-2297 (part 3): refactor

[kudu-CR] KUDU-2297 (part 4): periodically dump stacks to diagnostics log

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9330 ) Change subject: KUDU-2297 (part 4): periodically dump stacks to diagnostics log .. KUDU-2297 (part 4): periodically dump stacks

[kudu-CR] Don't perform compactions when clean time has not been advanced

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has removed a vote on this change. Change subject: Don't perform compactions when clean time has not been advanced .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] Don't perform compactions when clean time has not been advanced

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9436 ) Change subject: Don't perform compactions when clean time has not been advanced .. Patch Set 1: Verified+1

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 12: (6 comments)

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Hao Hao (Code Review)
Hello Alex Rodoni, Mike Percy, Dan Burkert, David Ribeiro Alves, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8823 to look at the new patch set (#13). Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode

[kudu-CR] KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks

2018-02-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9329 ) Change subject: KUDU-2297 (part 3): refactor process-wide stack collection out of /stacks .. Patch Set 8: Code-Review+2 -- To view,

[kudu-CR] Don't perform compactions when clean time has not been advanced

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9436 ) Change subject: Don't perform compactions when clean time has not been advanced .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 8: (1 comment)

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 2: (1 comment)

[kudu-CR] Optionally advance safe time with non-write operations

2018-02-23 Thread David Ribeiro Alves (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9435 to review the following change. Change subject: Optionally advance safe time with non-write operations .. Optionally

[kudu-CR] Don't perform compactions when clean time has not been advanced

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9436 Change subject: Don't perform compactions when clean time has not been advanced .. Don't perform compactions when clean time has

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 6: (1 comment)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9373 to look at the new patch set (#8). Change subject: KUDU-2319 follower masters should be able to verify authn tokens

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 5: (5 comments)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9373 to look at the new patch set (#7). Change subject: KUDU-2319 follower masters should be able to verify authn tokens

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9433 ) Change subject: [util] fix build on OS X .. Patch Set 1: > Build Started http://jenkins.kudu.apache.org/job/kudu-gerrit/12197/ IWYU

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9433 ) Change subject: [util] fix build on OS X .. [util] fix build on OS X This is a follow-up for

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9433 ) Change subject: [util] fix build on OS X .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/9433 To

[kudu-CR] KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8692 ) Change subject: KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client .. KUDU-2191 (5/n): Add Kerberos SASL support to the

[kudu-CR] KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8692 ) Change subject: KUDU-2191 (5/n): Add Kerberos SASL support to the HMS client .. Patch Set 11: Code-Review+2 -- To view, visit

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 2: (10 comments)

[kudu-CR] KUDU-2259: add real user to AuthenticationCredentialsPB

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9374 ) Change subject: KUDU-2259: add real user to AuthenticationCredentialsPB .. Patch Set 2: (6 comments)

[kudu-CR] KUDU-2291 (part 8): fix a TSAN issue with libunwind initialization

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9409 ) Change subject: KUDU-2291 (part 8): fix a TSAN issue with libunwind initialization .. KUDU-2291 (part 8): fix a TSAN issue with

[kudu-CR] KUDU-2291 (part 8): fix a TSAN issue with libunwind initialization

2018-02-23 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9409 ) Change subject: KUDU-2291 (part 8): fix a TSAN issue with libunwind initialization .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2309: /masters can show the wrong list of masters

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9378 ) Change subject: KUDU-2309: /masters can show the wrong list of masters .. Patch Set 3: (1 comment)

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9433 ) Change subject: [util] fix build on OS X .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/9433 To

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 10: (2 comments)

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9433 ) Change subject: [util] fix build on OS X .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9433 To

[kudu-CR](branch-1.6.x) KUDU-1613: evict replicas with wrong server UUIDs

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9410 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 1: (1 comment)

[kudu-CR] [util] fix build on OS X

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9433 Change subject: [util] fix build on OS X .. [util] fix build on OS X This is a follow-up for c20b84bb600575e4f6a34846c07acf4603b1bf26.

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Grant Henke (Code Review)
Hello Alex Rodoni, Dan Burkert, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9432 to look at the new patch set (#2). Change subject: [doc] Document the new decimal column type

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 1: (1 comment)

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9432 ) Change subject: [doc] Document the new decimal column type .. Patch Set 1: (1 comment)

[kudu-CR] [doc] Document the new decimal column type

2018-02-23 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9432 Change subject: [doc] Document the new decimal column type .. [doc] Document the new decimal column type Change-Id:

[kudu-CR] webserver-stress-itest: fix flakiness

2018-02-23 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9414 to look at the new patch set (#3). Change subject: webserver-stress-itest: fix flakiness ..

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 12: (3 comments)

[kudu-CR] KUDU-2325. Fix CHECK failure in RevokeSigData()

2018-02-23 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9430 to review the following change. Change subject: KUDU-2325. Fix CHECK failure in RevokeSigData() .. KUDU-2325. Fix

[kudu-CR] java: improve error messages when tokens are not used

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9070 ) Change subject: java: improve error messages when tokens are not used .. Patch Set 1: yep, this is on my list today to revise -- To

[kudu-CR] java: improve error messages when tokens are not used

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9070 ) Change subject: java: improve error messages when tokens are not used .. Patch Set 1: (1 comment) > > Would be great to get this in

[kudu-CR] fs: clarify error message at startup

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9281 ) Change subject: fs: clarify error message at startup .. fs: clarify error message at startup Currently the error messages you

[kudu-CR] KUDU-1704: add c++ client support for READ YOUR WRITES mode

2018-02-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8823 ) Change subject: KUDU-1704: add c++ client support for READ_YOUR_WRITES mode .. Patch Set 10: (1 comment)

[kudu-CR] fs: clarify error message at startup

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9281 ) Change subject: fs: clarify error message at startup .. Patch Set 5: Code-Review+2 Carrying forward Will's +2 -- To view, visit

[kudu-CR] [kudu-jepsen] clarify on how binaries get to DB nodes

2018-02-23 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/9419 ) Change subject: [kudu-jepsen] clarify on how binaries get to DB nodes .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 6: (5 comments)

[kudu-CR] [Spark] Simplifiy creating range partitioned tables

2018-02-23 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9379 ) Change subject: [Spark] Simplifiy creating range partitioned tables .. [Spark] Simplifiy creating range partitioned tables In

[kudu-CR] KUDU-2290: Tool to re-create a tablet

2018-02-23 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9393 ) Change subject: KUDU-2290: Tool to re-create a tablet .. Patch Set 2: (1 comment)

[kudu-CR] webserver-stress-itest: fix flakiness

2018-02-23 Thread Todd Lipcon (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9414 to look at the new patch set (#2). Change subject: webserver-stress-itest: fix flakiness ..

[kudu-CR] KUDU-2319 follower masters should be able to verify authn tokens

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9373 ) Change subject: KUDU-2319 follower masters should be able to verify authn tokens .. Patch Set 5: (1 comment)

[kudu-CR] [java] fix the regression test for KUDU-2267/KUDU-2319

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9388 ) Change subject: [java] fix the regression test for KUDU-2267/KUDU-2319 .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [kudu-jepsen] clarify on how binaries get to DB nodes

2018-02-23 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9419 to look at the new patch set (#2). Change subject: [kudu-jepsen] clarify on how binaries get to DB nodes ..

[kudu-CR] [kudu-jepsen] clarify on how binaries

2018-02-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9419 Change subject: [kudu-jepsen] clarify on how binaries .. [kudu-jepsen] clarify on how binaries Added a note on how Kudu binaries get to

[kudu-CR] [Spark] Simplifiy creating range partitioned tables

2018-02-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9379 ) Change subject: [Spark] Simplifiy creating range partitioned tables .. Patch Set 1: Code-Review+2 -- To view, visit