[kudu-CR] KUDU-2254: Detect and warn about misusage of KuduContext

2018-01-10 Thread Hao Hao (Code Review)
Hao Hao has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9004 Change subject: KUDU-2254: Detect and warn about misusage of KuduContext .. KUDU-2254: Detect and warn about misusage of KuduContext Since Kud

[kudu-CR] [tls socket-test] fix test to pass on Ubuntu 16.04

2018-01-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8996 to look at the new patch set (#4). Change subject: [tls_socket-test] fix test to pass on Ubuntu 16.04 ..

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8978 to look at the new patch set (#4). Change subject: KUDU-2202: support for removing data directories (take two)

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8978 ) Change subject: KUDU-2202: support for removing data directories (take two) .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/

[kudu-CR] [tls socket-test] fix test to pass at Ubuntu 16.04

2018-01-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8996 to look at the new patch set (#3). Change subject: [tls_socket-test] fix test to pass at Ubuntu 16.04 ..

[kudu-CR] [tls socket-test] fix test to pass at Ubuntu 16.04

2018-01-10 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8996 to look at the new patch set (#2). Change subject: [tls_socket-test] fix test to pass at Ubuntu 16.04 ..

[kudu-CR] KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8830 ) Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client) .. Patch Set 8: (23 comments) I think this could do with

[kudu-CR] [webui] Make tombstone tablet info less confusing

2018-01-10 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8981 ) Change subject: [webui] Make tombstone tablet info less confusing .. [webui] Make tombstone tablet info less confusing Previous

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8995 ) Change subject: fs: update default data dir group size .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/8995/3//COMMIT_M

[kudu-CR] KUDU-2236: use debug logging where appropriate

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8961 ) Change subject: KUDU-2236: use debug logging where appropriate .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/8961/2/j

[kudu-CR] [docs] Added steps to update HMS after migrating to multiple Kudu masters

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8948 ) Change subject: [docs] Added steps to update HMS after migrating to multiple Kudu masters .. Patch Set 8: (3 comments) http://gerrit.cl

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8995 ) Change subject: fs: update default data dir group size .. Patch Set 3: > Patch Set 2: > > Interesting that this change is causing build f

[kudu-CR] [docs] Document how to recover from a majority failed tablet

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8402 ) Change subject: [docs] Document how to recover from a majority failed tablet .. Patch Set 5: (4 comments) This doc is nice. I wonder if

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8995 to look at the new patch set (#3). Change subject: fs: update default data dir group size ..

[kudu-CR] [webui] Make tombstone tablet info less confusing

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8981 ) Change subject: [webui] Make tombstone tablet info less confusing .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera

[kudu-CR] [tls socket-test] fix test to run smoothly at kernel 4.4.x

2018-01-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8996 ) Change subject: [tls_socket-test] fix test to run smoothly at kernel 4.4.x .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8978 ) Change subject: KUDU-2202: support for removing data directories (take two) .. Patch Set 3: (5 comments) Just a couple more nits from me

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Adar Dembo (Code Review)
Hello Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8978 to look at the new patch set (#3). Change subject: KUDU-2202: support for removing data directories (take two) ..

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8978 ) Change subject: KUDU-2202: support for removing data directories (take two) .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8995 ) Change subject: fs: update default data dir group size .. Patch Set 2: Interesting that this change is causing build failures of the rand

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8995 ) Change subject: fs: update default data dir group size .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8995/2/src/kudu/f

[kudu-CR] data dirs: fix logging message

2018-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8999 ) Change subject: data_dirs: fix logging message .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8999 To un

[kudu-CR] data dirs: fix logging message

2018-01-10 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8999 to look at the new patch set (#2). Change subject: data_dirs: fix logging message .. data_dirs: fix logg

[kudu-CR] data dirs: fix logging message

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8999 ) Change subject: data_dirs: fix logging message .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/8999/1//COMMIT_MSG Commi

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8995 ) Change subject: fs: update default data dir group size .. Patch Set 2: Code-Review+1 (1 comment) Since this change has non-trivial impact

[kudu-CR] data dirs: fix logging message

2018-01-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8999 ) Change subject: data_dirs: fix logging message .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/8999/1//COMMIT_MSG Commit

[kudu-CR] data dirs: fix logging message

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8999 Change subject: data_dirs: fix logging message .. data_dirs: fix logging message If fs_target_data_dirs_per_tablet is set to be greater th

[kudu-CR] [tls socket-test] fix test to run smoothly at kernel 4.4.x

2018-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8996 Change subject: [tls_socket-test] fix test to run smoothly at kernel 4.4.x .. [tls_socket-test] fix test to run smoothly at kernel 4.4.x

[kudu-CR] fs: update default data dir group size

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8995 Change subject: fs: update default data dir group size .. fs: update default data dir group size This patch makes fs_target_data_dirs_per_

[kudu-CR] [webui] Make tombstone tablet info less confusing

2018-01-10 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8981 to look at the new patch set (#3). Change subject: [webui] Make tombstone tablet info less confusing

[kudu-CR] [webui] Make tombstone tablet info less confusing

2018-01-10 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/8981 ) Change subject: [webui] Make tombstone tablet info less confusing .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/898

[kudu-CR] KUDU-2202: support for removing data directories (take two)

2018-01-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8978 ) Change subject: KUDU-2202: support for removing data directories (take two) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/

[kudu-CR] KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)

2018-01-10 Thread Grant Henke (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/8830 to look at the new patch set (#8). Change subject: KUDU-721: Support for Decimal type, Part 1 (Server, C++ Client)