[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4134/2/src/kudu/tools/ts-cli.cc File src/kud

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 4: (5 comments) TFTR Alexey/Todd, please see response inlined. http://gerrit.cloudera.org:8080

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/3127/ -- To view, visit http://gerrit.

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4134 to look at the new patch set (#4). Change subject: Fix kudu-ts-cli crash when there is no data in tablet ..

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3126/ -- To view, visit http://gerrit.

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Todd Lipcon, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4134 to look at the new patch set (#3). Change subject: Fix kudu-ts-cli crash when there is no data in tablet ..

[kudu-CR] KUDU-1048 master should show versions of tservers, version summary

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1048 master should show versions of tservers, version summary .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4104/2/src/kudu/master/ts_mana

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tracing: gzip the trace JSON ajax response .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3125/ -- To view, visit http://gerrit.cloudera.or

[kudu-CR] tracing: gzip the trace JSON ajax response

2016-08-28 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4154 to review the following change. Change subject: tracing: gzip the trace JSON ajax response .. tracing: gzip the trace J

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4147 to look at the new patch set (#3). Change subject: KUDU-687: use client in ksck for master operations .. KUDU-687:

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-687: use client in ksck for master operations .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3124/ -- To view, visit http://gerrit.clo

[kudu-CR] KUDU-687: expose additional tablet metadata in C++ client

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-687: expose additional tablet metadata in C++ client .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3119/ -- To view, visit http://ger

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4148 to look at the new patch set (#3). Change subject: tool: split up action descriptions .. tool: split up action desc

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4151 to look at the new patch set (#2). Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. tool: port cfil

[kudu-CR] cfile: replace DumpIteratorOptions with number of rows

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: cfile: replace DumpIteratorOptions with number of rows .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3121/ -- To view, visit http://gerrit

[kudu-CR] KUDU-687: expose additional tablet metadata in C++ client

2016-08-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4146 to look at the new patch set (#2). Change subject: KUDU-687: expose additional tablet metadata in C++ client .. KUD

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-687: use client in ksck for master operations .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3120/ -- To view, visit http://gerrit.clo

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: split up action descriptions .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3123/ -- To view, visit http://gerrit.cloudera.org:8080/4

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4147 to look at the new patch set (#2). Change subject: KUDU-687: use client in ksck for master operations .. KUDU-687:

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3122/ -- To view, visit http://gerrit.cloudera

[kudu-CR] KUDU-687: expose additional tablet metadata in C++ client

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-687: expose additional tablet metadata in C++ client .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/4146/1/src/kudu/client/client.h File src

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4151/1/src/kudu/tools/tool_action_fs.cc File src/kudu

[kudu-CR] cfile: replace DumpIteratorOptions with number of rows

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: cfile: replace DumpIteratorOptions with number of rows .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4150/1/src/kudu/cfile/cfile-dump.cc File sr

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: tool: split up action descriptions .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4148/2/src/kudu/tools/kudu-tool-test.cc File src/kudu/tools/kudu

[kudu-CR] KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/3823/15/src/kudu/tablet/tab

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-687: use client in ksck for master operations .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4147/1//COMMIT_MSG Commit Message: Line 17:

[kudu-CR] KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

2016-08-28 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata .. Patch Set 15: (1 comment) http://gerrit.cloudera.org:8080/#/c/3823/15/src/kudu/tablet/tab

[kudu-CR] KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata .. Patch Set 16: Build Started http://104.196.14.100/job/kudu-gerrit/3118/ -- To view, vi

[kudu-CR] KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

2016-08-28 Thread Dinesh Bhat (Code Review)
Hello Dan Burkert, Mike Percy, Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3823 to look at the new patch set (#16). Change subject: KUDU-1500: Data race in RaftConsensusITest.TestCorruptReplicaMetadata

[kudu-CR] Fix kudu-ts-cli crash when there is no data in tablet

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Fix kudu-ts-cli crash when there is no data in tablet .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4134/2/src/kudu/tools/ts-cli.cc File src/kud

[kudu-CR] [docs] added tip on generic git commit guidelines

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: [docs] added tip on generic git commit guidelines .. [docs] added tip on generic git commit guidelines Change-Id: I5493723f7f0b337b73e3ef5ed4a7621fe524992

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4151/1/src/kudu/tools/tool_action_fs.cc File src/kud

[kudu-CR] cfile: replace DumpIteratorOptions with number of rows

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: cfile: replace DumpIteratorOptions with number of rows .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/4150/1/src/kudu/cfile/cfile-dump.cc File s

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: tool: split up action descriptions .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4148/2/src/kudu/tools/kudu-tool-test.cc File src/kudu/tools/kud

[kudu-CR] KUDU-687: use client in ksck for master operations

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-687: use client in ksck for master operations .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4147/1//COMMIT_MSG Commit Message: Line 17:

[kudu-CR] KUDU-687: expose additional tablet metadata in C++ client

2016-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-687: expose additional tablet metadata in C++ client .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/4146/1/src/kudu/client/client.h File sr

[kudu-CR] compaction policy: fix bound calculation

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: compaction_policy: fix bound calculation .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3116/ -- To view, visit http://gerrit.cloudera.org:

[kudu-CR] compaction policy: fix bound calculation

2016-08-28 Thread Todd Lipcon (Code Review)
Hello Anonymous Coward #174, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4152 to review the following change. Change subject: compaction_policy: fix bound calculation .. compaction_polic

[kudu-CR] KUDU-1582. Optimize budgeted compaction policy with an approximation

2016-08-28 Thread Todd Lipcon (Code Review)
Hello Anonymous Coward #174, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4153 to review the following change. Change subject: KUDU-1582. Optimize budgeted compaction policy with an approximation ...

[kudu-CR] KUDU-1582. Optimize budgeted compaction policy with an approximation

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1582. Optimize budgeted compaction policy with an approximation .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3117/ -- To view, visi

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4148 to look at the new patch set (#2). Change subject: tool: split up action descriptions .. tool: split u

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4151 to review the following change. Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. tool: port cfile-dump

[kudu-CR] tool: port cfile-dump to 'kudu fs dump cfile'

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: port cfile-dump to 'kudu fs dump_cfile' .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3115/ -- To view, visit http://gerrit.cloudera

[kudu-CR] cfile: replace DumpIteratorOptions with number of rows

2016-08-28 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4150 to review the following change. Change subject: cfile: replace DumpIteratorOptions with number of rows .. cfile: repla

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: split up action descriptions .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3113/ -- To view, visit http://gerrit.cloudera.org:8080/4

[kudu-CR] cfile: replace DumpIteratorOptions with number of rows

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: cfile: replace DumpIteratorOptions with number of rows .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3114/ -- To view, visit http://gerrit

[kudu-CR] KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when Spark expects to receive Array[Byte], so change is to return

2016-08-28 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1581 Fix DataFrame read failure when table has Binary Col For Binary Cols, kudu-spark is returning a ByteBuffer object when Spark expects to receive Array[Byte], so change is to return a copy of the byte array. Modified test

[kudu-CR](gh-pages) kudu flume sink blog post

2016-08-28 Thread Ara Ebrahimi (Code Review)
Ara Ebrahimi has uploaded a new change for review. http://gerrit.cloudera.org:8080/4149 Change subject: kudu flume sink blog post .. kudu flume sink blog post Change-Id: I33166f0c3a0f739527d009808d4433342f9a95a8 --- M _posts/

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: tool: split up action descriptions .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/3112/ -- To view, visit http://gerrit.cloudera.org:8080/4

[kudu-CR] tool: split up action descriptions

2016-08-28 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4148 to review the following change. Change subject: tool: split up action descriptions .. tool: split up action descriptio