[kudu-CR] [java] small fixes for diff scan setup logic

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13553 to review the following change. Change subject: [java] small fixes for diff scan setup logic

[kudu-CR] KUDU-2809 (5/6): add diff scan support to fuzz-itest

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Tidy Bot, Mike Percy, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13537 to look at the new patch set (#4). Change subject: KUDU-2809 (5/6): add diff scan support to fuzz-itest

[kudu-CR] fuzz-itest: assorted cleanup

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13532 ) Change subject: fuzz-itest: assorted cleanup .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2809 (3/6): C++ private API for diff scans

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13535 ) Change subject: KUDU-2809 (3/6): C++ private API for diff scans .. Patch Set 3: (4 comments)

[kudu-CR] schema: cache first is deleted virtual column index

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, Grant Henke, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13554 to review the following change. Change subject: schema: cache first is_deleted virtual column index

[kudu-CR] KUDU-2809 (2/6): skip unobservable rows when iterating an MRS

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13534 ) Change subject: KUDU-2809 (2/6): skip unobservable rows when iterating an MRS .. Patch Set 2: (2 comments)

[kudu-CR] KUDU-2809 (3/6): C++ private API for diff scans

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13535 to look at the new patch set (#3). Change subject: KUDU-2809 (3/6): C++ private API for diff scans

[kudu-CR] KUDU-2809 (2/6): skip unobservable rows when iterating an MRS

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13534 to look at the new patch set (#3). Change subject: KUDU-2809 (2/6): skip unobservable rows when iterating an MRS

[kudu-CR] fuzz-itest: assorted cleanup

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Tidy Bot, Mike Percy, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13532 to look at the new patch set (#3). Change subject: fuzz-itest: assorted cleanup

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/13341

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Todd Lipcon (Code Review)
Hello Thomas Marshall, Tidy Bot, Lars Volker, Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13341 to look at the new patch set (#5). Change subject: Support SPNEGO for web server

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 3: (3 comments)

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 4: Code-Review+1 (4 comments) I didn't look deep at this patch, but

[kudu-CR] Add seek before mode for CBTree to accelerate CheckRowDeleted in dms.

2019-06-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13516 ) Change subject: Add seek before mode for CBTree to accelerate CheckRowDeleted in dms. .. Patch Set 2: (9 comments)

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 2: Verified+1 (1 comment)

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13552 ) Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Patch Set 2: Unrelated flake. -- To

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has removed a vote on this change. Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] [rebalance] Add '--known unhealthy tservers' flag to rebalancer

2019-06-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--known_unhealthy_tservers' flag to rebalancer .. Patch Set 1: (1 comment)

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Todd Lipcon (Code Review)
Hello Thomas Marshall, Tidy Bot, Lars Volker, Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13341 to look at the new patch set (#4). Change subject: Support SPNEGO for web server

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 3: (15 comments)

[kudu-CR] [backup] Make KuduBackupCLI executable

2019-06-06 Thread Grant Henke (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13551 to look at the new patch set (#2). Change subject: [backup] Make KuduBackupCLI executable

[kudu-CR] Support SPNEGO for web server

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13341 ) Change subject: Support SPNEGO for web server .. Patch Set 3: (15 comments)

[kudu-CR] [backup] Make KuduBackupCLI format optional

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [backup] Make KuduBackupCLI format optional .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13551 To unsubscribe,

[kudu-CR] [backup] Make KuduBackupCLI format optional

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13551 ) Change subject: [backup] Make KuduBackupCLI format optional .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] [backup] Fix backup CLI dependencies

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [backup] Fix backup CLI dependencies .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13547 To unsubscribe, visit

[kudu-CR] [backup] Fix backup CLI dependencies

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13547 ) Change subject: [backup] Fix backup CLI dependencies .. Patch Set 2: Verified+1 (2 comments)

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-06 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13552 to look at the new patch set (#2). Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

[kudu-CR] KUDU-2809 (3/6): C++ private API for diff scans

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13535 ) Change subject: KUDU-2809 (3/6): C++ private API for diff scans .. Patch Set 2: Code-Review+1 (3 comments) looks good modulo a couple

[kudu-CR] KUDU-2809 (2/6): skip unobservable rows when iterating an MRS

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13534 ) Change subject: KUDU-2809 (2/6): skip unobservable rows when iterating an MRS .. Patch Set 2: Code-Review+2 (2 comments)

[kudu-CR] sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13552 to review the following change. Change subject: sentry: allow caching of COLUMN/TABLE privileges when checking higher scopes

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 3: (1 comment)

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-06 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13549 to look at the new patch set (#3). Change subject: sentry: avoid authorizing every table in ListTables

[kudu-CR] [backup] Make KuduBackupCLI format optional

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13551 Change subject: [backup] Make KuduBackupCLI format optional .. [backup] Make KuduBackupCLI format optional This patch changes the

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13549 to look at the new patch set (#2). Change subject: sentry: avoid authorizing every table in ListTables

[kudu-CR] [backup] Fix backup CLI dependencies

2019-06-06 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13547 to look at the new patch set (#2). Change subject: [backup] Fix backup CLI dependencies .. [backup]

[kudu-CR] KUDU-2785: Add splitSizeBytes to the backup job

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13511 ) Change subject: KUDU-2785: Add splitSizeBytes to the backup job .. KUDU-2785: Add splitSizeBytes to the backup job This patch

[kudu-CR] sentry: don't send requests for DATABASE/SERVER privileges

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13494 ) Change subject: sentry: don't send requests for DATABASE/SERVER privileges .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] sentry: don't send requests for DATABASE/SERVER privileges

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13494 ) Change subject: sentry: don't send requests for DATABASE/SERVER privileges .. sentry: don't send requests for DATABASE/SERVER

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. hms: allow for tooling to run without Kudu plugin Currently,

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2809 (1/6): temporarily disable TestKuduScanner.testDiffScan

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13533 ) Change subject: KUDU-2809 (1/6): temporarily disable TestKuduScanner.testDiffScan .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] fuzz-itest: assorted cleanup

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13532 ) Change subject: fuzz-itest: assorted cleanup .. Patch Set 2: Code-Review+1 (1 comment)

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13549 ) Change subject: sentry: avoid authorizing every table in ListTables .. Patch Set 1: (1 comment)

[kudu-CR] sentry: avoid authorizing every table in ListTables

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13549 Change subject: sentry: avoid authorizing every table in ListTables .. sentry: avoid authorizing every table in ListTables Currently

[kudu-CR] memrowset: skip all uncommitted mutations when iterating

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13531 ) Change subject: memrowset: skip all uncommitted mutations when iterating .. memrowset: skip all uncommitted mutations when

[kudu-CR] memrowset: skip all uncommitted mutations when iterating

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13531 ) Change subject: memrowset: skip all uncommitted mutations when iterating .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [backup] Fix backup CLI dependencies

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13547 ) Change subject: [backup] Fix backup CLI dependencies .. Patch Set 1: Code-Review+1 (2 comments) Thanks for doing this!

[kudu-CR] KUDU-2785: Add splitSizeBytes to the backup job

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13511 ) Change subject: KUDU-2785: Add splitSizeBytes to the backup job .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] [backup] Fix backup CLI dependencies

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13547 Change subject: [backup] Fix backup CLI dependencies .. [backup] Fix backup CLI dependencies This patch fixes the backup dependencies in

[kudu-CR] KUDU-2809 (2/6): skip unobservable rows when iterating an MRS

2019-06-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/13534 ) Change subject: KUDU-2809 (2/6): skip unobservable rows when iterating an MRS .. Patch Set 2: Code-Review+1 -- To view, visit

[kudu-CR] sentry: don't send requests for DATABASE/SERVER privileges

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13494 to look at the new patch set (#7). Change subject: sentry: don't send requests for DATABASE/SERVER privileges

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13510 to look at the new patch set (#6). Change subject: hms: allow for tooling to run without Kudu plugin

[kudu-CR] ksck remote-test: make checksum tests more robust

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13529 ) Change subject: ksck_remote-test: make checksum tests more robust .. ksck_remote-test: make checksum tests more robust They

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 5: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2809 (1/6): temporarily disable TestKuduScanner.testDiffScan

2019-06-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/13533 ) Change subject: KUDU-2809 (1/6): temporarily disable TestKuduScanner.testDiffScan .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] ksck remote-test: make checksum tests more robust

2019-06-06 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: ksck_remote-test: make checksum tests more robust .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13529 To

[kudu-CR] ksck remote-test: make checksum tests more robust

2019-06-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/13529 ) Change subject: ksck_remote-test: make checksum tests more robust .. Patch Set 1: Verified+1 Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2830: [java] Improve RPC traces on RPC timeout

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13509 ) Change subject: KUDU-2830: [java] Improve RPC traces on RPC timeout .. KUDU-2830: [java] Improve RPC traces on RPC timeout

[kudu-CR] KUDU-2830: [java] Improve RPC traces on RPC timeout

2019-06-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/13509 ) Change subject: KUDU-2830: [java] Improve RPC traces on RPC timeout .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2809 (6/6): reenable and update TestKuduScanner.testDiffScan

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13538 ) Change subject: KUDU-2809 (6/6): reenable and update TestKuduScanner.testDiffScan .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2809 (3/6): C++ private API for diff scans

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13535 ) Change subject: KUDU-2809 (3/6): C++ private API for diff scans .. Patch Set 2: (2 comments)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 5: (1 comment)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13510 to look at the new patch set (#5). Change subject: hms: allow for tooling to run without Kudu plugin

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 4: (1 comment)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 3: (1 comment)

[kudu-CR] fuzz-itest: assorted cleanup

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13532 ) Change subject: fuzz-itest: assorted cleanup .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/13532 To

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 4: (5 comments)

[kudu-CR] rpc-test: deflake TestClientConnectionMetrics

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13543 ) Change subject: rpc-test: deflake TestClientConnectionMetrics .. Patch Set 1: Verified+1 Overriding Jenkins, unrelated test flake. --

[kudu-CR] rpc-test: deflake TestClientConnectionMetrics

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13543 ) Change subject: rpc-test: deflake TestClientConnectionMetrics .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13510 to look at the new patch set (#4). Change subject: hms: allow for tooling to run without Kudu plugin

[kudu-CR] rpc-test: deflake TestClientConnectionMetrics

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13543 ) Change subject: rpc-test: deflake TestClientConnectionMetrics .. rpc-test: deflake TestClientConnectionMetrics Looks like

[kudu-CR] KUDU-2785: Add splitSizeBytes to the backup job

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13511 ) Change subject: KUDU-2785: Add splitSizeBytes to the backup job .. Patch Set 4: (2 comments)

[kudu-CR] KUDU-2785: Add splitSizeBytes to the backup job

2019-06-06 Thread Grant Henke (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13511 to look at the new patch set (#5). Change subject: KUDU-2785: Add splitSizeBytes to the backup job

[kudu-CR] KUDU-2785: Add splitSizeBytes to the backup job

2019-06-06 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/13511 ) Change subject: KUDU-2785: Add splitSizeBytes to the backup job .. Patch Set 4: (2 comments)

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. Patch Set 1: (1 comment)

[kudu-CR] rpc-test: deflake TestClientConnectionMetrics

2019-06-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13543 ) Change subject: rpc-test: deflake TestClientConnectionMetrics .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [rebalance] Add '--known unhealthy tservers' flag to rebalancer

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--known_unhealthy_tservers' flag to rebalancer .. Patch Set 1: (2 comments)

[kudu-CR] [rebalance] Add '--known unhealthy tservers' flag to rebalancer

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--known_unhealthy_tservers' flag to rebalancer .. Patch Set 1: (6 comments)

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13540/1//COMMIT_MSG

[kudu-CR] rpc-test: deflake TestClientConnectionMetrics

2019-06-06 Thread Adar Dembo (Code Review)
Hello Will Berkeley, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/13543 to review the following change. Change subject: rpc-test: deflake TestClientConnectionMetrics ..

[kudu-CR] [rebalance] Add '--known unhealthy tservers' flag to rebalancer

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13539 ) Change subject: [rebalance] Add '--known_unhealthy_tservers' flag to rebalancer .. Patch Set 1: (2 comments) Thank you for your

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 2: (1 comment)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 3: (1 comment)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 3: (1 comment)

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [hms] Adjust storage handler package .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/13540 To unsubscribe, visit

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 3: (1 comment)

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Andrew Wong (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Grant Henke, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13510 to look at the new patch set (#3). Change subject: hms: allow for tooling to run without Kudu plugin

[kudu-CR] hms: allow for tooling to run without Kudu plugin

2019-06-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13510 ) Change subject: hms: allow for tooling to run without Kudu plugin .. Patch Set 2: (1 comment)

[kudu-CR] [java] Add newComparisonPredicate Object API

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13527 ) Change subject: [java] Add newComparisonPredicate Object API .. [java] Add newComparisonPredicate Object API Adds a

[kudu-CR] [java] Add PartialRow addObject API

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13528 ) Change subject: [java] Add PartialRow addObject API .. [java] Add PartialRow addObject API Adds a new API to add an Object to a

[kudu-CR] KUDU-2809 (5/6): add diff scan support to fuzz-itest

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13537 ) Change subject: KUDU-2809 (5/6): add diff scan support to fuzz-itest .. Patch Set 3: Verified+1 Overriding Jenkins, unrelated

[kudu-CR] KUDU-2809 (5/6): add diff scan support to fuzz-itest

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/13537 ) Change subject: KUDU-2809 (5/6): add diff scan support to fuzz-itest .. Removed reviewer Kudu Jenkins with the following votes:

[kudu-CR] fuzz-itest: assorted cleanup

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13532 ) Change subject: fuzz-itest: assorted cleanup .. Patch Set 2: (1 comment)

[kudu-CR] [java] Add PartialRow addObject API

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13528 ) Change subject: [java] Add PartialRow addObject API .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [java] Add newComparisonPredicate Object API

2019-06-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13527 ) Change subject: [java] Add newComparisonPredicate Object API .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [hms] Adjust storage handler package

2019-06-06 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13540 ) Change subject: [hms] Adjust storage handler package .. Patch Set 1: This should be coordinated with

  1   2   >