[kudu-CR] WIP [tools] first draft of rebalancing in kudu CLI

2018-05-16 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10399 to look at the new patch set (#4). Change subject: WIP [tools] first draft of rebalancing in kudu CLI

[kudu-CR] [consensus] KUDU-2443 fix replica replacement of RF=1

2018-05-16 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10438 to look at the new patch set (#2). Change subject: [consensus] KUDU-2443 fix replica replacement of RF=1

[kudu-CR] KUDU-2427: make ksck-test replica order deterministic

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10433 ) Change subject: KUDU-2427: make ksck-test replica order deterministic .. KUDU-2427: make ksck-test replica order deterministic

[kudu-CR] KUDU-2427: make ksck-test replica order deterministic

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10433 ) Change subject: KUDU-2427: make ksck-test replica order deterministic .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2427: update to latest version of curl

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10432 ) Change subject: KUDU-2427: update to latest version of curl .. Patch Set 2: Code-Review+2 Carrying forward Alexey's +2. -- To view,

[kudu-CR] KUDU-2427: update to latest version of curl

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10432 ) Change subject: KUDU-2427: update to latest version of curl .. KUDU-2427: update to latest version of curl The existing version

[kudu-CR] KUDU-2443 moving single-replica tablets is broken

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10438 Change subject: KUDU-2443 moving single-replica tablets is broken .. KUDU-2443 moving single-replica tablets is broken Fixed bug in

[kudu-CR] KUDU-2191: Metadata Upgrade Tool

2018-05-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10075 ) Change subject: KUDU-2191: Metadata Upgrade Tool .. Patch Set 14: (2 comments)

[kudu-CR] KUDU-2191: Metadata Upgrade Tool

2018-05-16 Thread Hao Hao (Code Review)
Hello Tidy Bot, Alexey Serbin, Dan Burkert, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10075 to look at the new patch set (#14). Change subject: KUDU-2191: Metadata Upgrade Tool

[kudu-CR] [DOCS] Added an info on creating a table in Impala with a custom replication factor

2018-05-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10411 ) Change subject: [DOCS] Added an info on creating a table in Impala with a custom replication factor .. Patch Set 2: Hi Mike, Could you

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 10: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2427: adjust gold linker detection

2018-05-16 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10428 to look at the new patch set (#2). Change subject: KUDU-2427: adjust gold linker detection

[kudu-CR] build: only call find package on gperftools if we're going to use it

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10429 ) Change subject: build: only call find_package on gperftools if we're going to use it .. Patch Set 1: Code-Review+2 -- To view,

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 10: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 10: (1 comment) Thanks!

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10332 to look at the new patch set (#10). Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 9: > (1 comment) > > Almost there: just

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 9: (1 comment) Almost there: just one more

[kudu-CR] KUDU-2427: update to latest version of curl

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10432 ) Change subject: KUDU-2427: update to latest version of curl .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1867 Improve the "Could not lock .../block manager instance" error log

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Could not lock .../block_manager_instance" error log .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1867 Improve the "Could not lock .../block manager instance" error log

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Could not lock .../block_manager_instance" error log .. Patch Set 1: (1 comment) After a few

[kudu-CR] KUDU-1867 Improve the "Could not lock .../block manager instance" error log

2018-05-16 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10419 to look at the new patch set (#2). Change subject: KUDU-1867 Improve the "Could not lock .../block_manager_instance" error log

[kudu-CR] KUDU-2427: adjust gold linker detection

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10428 to review the following change. Change subject: KUDU-2427: adjust gold linker detection .. KUDU-2427: adjust gold

[kudu-CR] KUDU-2427: relax dlsym call to dl iterate phdr

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10434 to review the following change. Change subject: KUDU-2427: relax dlsym call to dl_iterate_phdr .. KUDU-2427: relax

[kudu-CR] KUDU-2427: fix glog symbolization in certain ASLR environments

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10431 to review the following change. Change subject: KUDU-2427: fix glog symbolization in certain ASLR environments ..

[kudu-CR] KUDU-2427: retry more system calls on EINTR

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10435 to review the following change. Change subject: KUDU-2427: retry more system calls on EINTR .. KUDU-2427: retry more

[kudu-CR] WIP: KUDU-1889: support openssl 1.1

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10436 to review the following change. Change subject: WIP: KUDU-1889: support openssl 1.1 .. WIP: KUDU-1889: support

[kudu-CR] KUDU-2427: only add -fno-sized-deallocation for C++ files

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10430 to review the following change. Change subject: KUDU-2427: only add -fno-sized-deallocation for C++ files ..

[kudu-CR] KUDU-2427: update breakpad to latest HEAD

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10427 to review the following change. Change subject: KUDU-2427: update breakpad to latest HEAD .. KUDU-2427: update

[kudu-CR] KUDU-2427: make ksck-test replica order deterministic

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10433 to review the following change. Change subject: KUDU-2427: make ksck-test replica order deterministic .. KUDU-2427:

[kudu-CR] KUDU-2427: update to latest version of curl

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10432 to review the following change. Change subject: KUDU-2427: update to latest version of curl .. KUDU-2427: update to

[kudu-CR] build: only call find package on gperftools if we're going to use it

2018-05-16 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10429 to review the following change. Change subject: build: only call find_package on gperftools if we're going to use it

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 8: (2 comments)

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10332 to look at the new patch set (#9). Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 12: (3 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#12). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 8: (2 comments) A couple of nits

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: (3 comments)

[kudu-CR] [java] Add kudu-client dep to kudu-spark-tools

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10425 ) Change subject: [java] Add kudu-client dep to kudu-spark-tools .. [java] Add kudu-client dep to kudu-spark-tools We use the

[kudu-CR] [java] Remove private method usage in kudu-mapreduce

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10424 ) Change subject: [java] Remove private method usage in kudu-mapreduce .. [java] Remove private method usage in kudu-mapreduce

[kudu-CR] [java] Remove private method usage in kudu-mapreduce

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10424 ) Change subject: [java] Remove private method usage in kudu-mapreduce .. Patch Set 1: (1 comment)

[kudu-CR] [java] Add kudu-client dep to kudu-spark-tools

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10425 ) Change subject: [java] Add kudu-client dep to kudu-spark-tools .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [java] Remove private method usage in kudu-mapreduce

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10424 ) Change subject: [java] Remove private method usage in kudu-mapreduce .. Patch Set 1: Code-Review+2 (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: Code-Review+1 The test failures all seem to be clock sync

[kudu-CR] [java] Expose the table replication factor on KuduTable

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10382 ) Change subject: [java] Expose the table replication factor on KuduTable .. [java] Expose the table replication factor on

[kudu-CR] [java] Expose the table replication factor on KuduTable

2018-05-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10382 ) Change subject: [java] Expose the table replication factor on KuduTable .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-1867 Improve the "Failed to load FS layout" error log

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Failed to load FS layout" error log .. Patch Set 1: It's there for both masters and tservers,

[kudu-CR] remove logged Kudu URLs

2018-05-16 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: remove logged Kudu URLs .. remove logged Kudu URLs Previously we would log urls like:

[kudu-CR] KUDU-1867 Improve the "Failed to load FS layout" error log

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Failed to load FS layout" error log .. Patch Set 1: > > Hi Will. Could you tell me how to do

[kudu-CR] KUDU-1867 Improve the "Failed to load FS layout" error log

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Failed to load FS layout" error log .. Patch Set 1: > Hi Will. Could you tell me how to do

[kudu-CR] KUDU-1867 Improve the "Failed to load FS layout" error log

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Failed to load FS layout" error log .. Patch Set 1: Hi Will. Could you tell me how to do

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 11: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#11). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] remove logged Kudu URLs

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: remove logged Kudu URLs .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9280 To

[kudu-CR] remove logged Kudu URLs

2018-05-16 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9280 to look at the new patch set (#7). Change subject: remove logged Kudu URLs .. remove

[kudu-CR] [java] Add kudu-client dep to kudu-spark-tools

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [java] Add kudu-client dep to kudu-spark-tools .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10425 To unsubscribe,

[kudu-CR] [java] Add kudu-client dep to kudu-spark-tools

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10425 ) Change subject: [java] Add kudu-client dep to kudu-spark-tools .. Patch Set 1: Verified+1 -- To view, visit

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 8: Code-Review+2 -- To view, visit

[kudu-CR] remove logged Kudu URLs

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: remove logged Kudu URLs .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9280 To

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 8: Just one-line change on curl_util.cc

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10332 to look at the new patch set (#8). Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths

[kudu-CR] [java] Add kudu-client dep to kudu-spark-tools

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10425 Change subject: [java] Add kudu-client dep to kudu-spark-tools .. [java] Add kudu-client dep to kudu-spark-tools We use the kudu-client

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 7: (1 comment)

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 7: (1 comment)

[kudu-CR] remove logged Kudu URLs

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: remove logged Kudu URLs .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9280 To

[kudu-CR] [java] Remove private method usage in kudu-mapreduce

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10424 Change subject: [java] Remove private method usage in kudu-mapreduce .. [java] Remove private method usage in kudu-mapreduce

[kudu-CR] KUDU-1867 Improve the "Failed to load FS layout" error log

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10419 ) Change subject: KUDU-1867 Improve the "Failed to load FS layout" error log .. Patch Set 1: (1 comment) I noticed another funny thing

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 7: (1 comment)

[kudu-CR] [java] Expose the table replication factor on KuduTable

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10382 ) Change subject: [java] Expose the table replication factor on KuduTable .. Patch Set 2: Verified+1 -- To view, visit

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 7: (1 comment)

[kudu-CR] [java] Expose the table replication factor on KuduTable

2018-05-16 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [java] Expose the table replication factor on KuduTable .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10382 To

[kudu-CR] KUDU-2035 Enable HTTP compression for all webserver's paths

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10332 ) Change subject: KUDU-2035 Enable HTTP compression for all webserver's paths .. Patch Set 7: (1 comment) LGTM. One nit / question.

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 10: (2 comments)

[kudu-CR] KUDU-2287 Expose election failures as metrics

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10076 ) Change subject: KUDU-2287 Expose election failures as metrics .. Patch Set 15: (1 comment) Also needs some tests, maybe in

[kudu-CR] KUDU-2287 Expose election failures as metrics

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10076 ) Change subject: KUDU-2287 Expose election failures as metrics .. Patch Set 15: Verified+1 Agreed, that's a known flake. -- To view,

[kudu-CR] KUDU-2287 Expose election failures as metrics

2018-05-16 Thread Will Berkeley (Code Review)
Will Berkeley has removed a vote on this change. Change subject: KUDU-2287 Expose election failures as metrics .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10076 To unsubscribe,

[kudu-CR] [java] Expose the table replication factor on KuduTable

2018-05-16 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10382 to look at the new patch set (#2). Change subject: [java] Expose the table replication factor on KuduTable

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 10: (1 comment)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#10). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10293 ) Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck .. Patch Set 9: (2 comments)

[kudu-CR] KUDU-2426 Fix WRONG SERVER UUID case in ksck

2018-05-16 Thread Attila Bukor (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10293 to look at the new patch set (#9). Change subject: KUDU-2426 Fix WRONG_SERVER_UUID case in ksck