[kudu-CR] KUDU-2249 give the TableRecordReader their own KuduClient to use.

2018-01-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8921 ) Change subject: KUDU-2249 give the TableRecordReader their own KuduClient to use. .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2191 (4/n): HMS Thrift client fault handling

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8494 ) Change subject: KUDU-2191 (4/n): HMS Thrift client fault handling .. Patch Set 7: (6 comments)

[kudu-CR] Add 'kudu fs list' tool

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8911 ) Change subject: Add 'kudu fs list' tool .. Patch Set 6: (10 comments)

[kudu-CR] periodic: fix a comment

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8931 ) Change subject: periodic: fix a comment .. periodic: fix a comment Change-Id: Id04bff4f6505595f60f26abe88de58cdded4d9d0

[kudu-CR] periodic: fix a comment

2018-01-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8931 ) Change subject: periodic: fix a comment .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8931 To

[kudu-CR] periodic: fix a comment

2018-01-03 Thread Adar Dembo (Code Review)
Hello Mike Percy, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8931 to review the following change. Change subject: periodic: fix a comment .. periodic: fix a comment

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

2018-01-03 Thread Alexey Serbin (Code Review)
Alexey Serbin 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 6: (12 comments) It seems I took a quick

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

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo 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 6: (4 comments) I only looked at the public

[kudu-CR] KUDU-2216. Post process gtest generated xml to include the output from the *.txt files

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8757 ) Change subject: KUDU-2216. Post process gtest generated xml to include the output from the *.txt files .. Patch Set 12: (2 comments)

[kudu-CR] KUDU-2229. consensus: Leader should not start FD

2018-01-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8711 ) Change subject: KUDU-2229. consensus: Leader should not start FD .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2249 give the TableRecordReader their own KuduClient to use.

2018-01-03 Thread Clemens Valiente (Code Review)
Clemens Valiente has posted comments on this change. ( http://gerrit.cloudera.org:8080/8921 ) Change subject: KUDU-2249 give the TableRecordReader their own KuduClient to use. .. Patch Set 1: (1 comment) Hi David, I