[kudu-CR] Refactor schema-design guide

2016-10-06 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4485 to look at the new patch set (#2). Change subject: Refactor schema-design guide .. Refactor schema-design guide

[kudu-CR] [java client] AsyncKuduClient#delayedSendRpcToTablet should return a Deferred

2016-10-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] AsyncKuduClient#delayedSendRpcToTablet should return a Deferred .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4654/1//COMMIT_MSG

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4648 to look at the new patch set (#4). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/4648/3/src/kudu/util/subprocess-test.cc File

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4648/2/src/kudu/util/subprocess-test.cc File

[kudu-CR] [flaky tests] Address SIGSEGV on ResultTracker while running alter table randomized-test

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [flaky tests] Address SIGSEGV on ResultTracker while running alter_table_randomized-test .. Patch Set 3: Code-Review+1 -- To view, visit

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/4648/2/src/kudu/util/subprocess-test.cc File

[kudu-CR] [flaky tests] Address SIGSEGV on ResultTracker while running alter table randomized-test

2016-10-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4629 to look at the new patch set (#3). Change subject: [flaky tests] Address SIGSEGV on ResultTracker while running alter_table_randomized-test

[kudu-CR] [java client] AsyncKuduClient#delayedSendRpcToTablet should return a Deferred

2016-10-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] AsyncKuduClient#delayedSendRpcToTablet should return a Deferred .. Patch Set 1: Verified+1 python test leakage -- To view, visit

[kudu-CR] log: add a test for race conditions

2016-10-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: log: add a test for race conditions .. Patch Set 2: well, turns out that this bug no longer exists after the relevant code was rewritten by

[kudu-CR] log: add a test for race conditions

2016-10-06 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4638 to look at the new patch set (#2). Change subject: log: add a test for race conditions .. log: add a

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/4648/2/src/kudu/util/pstack_watcher.cc File

[kudu-CR] KUDU-1649 - [python] Add Binary Type Support

2016-10-06 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has uploaded a new patch set (#2). Change subject: KUDU-1649 - [python] Add Binary Type Support .. KUDU-1649 - [python] Add Binary Type Support The Python client currently does not support the Binary type. This

[kudu-CR] KUDU-1649 - [python] Add Binary Type Support

2016-10-06 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has uploaded a new change for review. http://gerrit.cloudera.org:8080/4653 Change subject: KUDU-1649 - [python] Add Binary Type Support .. KUDU-1649 - [python] Add Binary Type Support The Python client

[kudu-CR] [c++client/samples] added README for the sample

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++client/samples] added README for the sample .. Patch Set 3: (23 comments) http://gerrit.cloudera.org:8080/#/c/4517/3/src/kudu/client/samples/README.adoc

[kudu-CR] [c++client/samples] added README for the sample

2016-10-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4517 to look at the new patch set (#6). Change subject: [c++client/samples] added README for the sample ..

[kudu-CR] [c++client/samples] added README for the sample

2016-10-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4517 to look at the new patch set (#5). Change subject: [c++client/samples] added README for the sample ..

[kudu-CR] [c++client/samples] added README for the sample

2016-10-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4517 to look at the new patch set (#4). Change subject: [c++client/samples] added README for the sample ..

[kudu-CR] KUDU-1681: DNS resolution failure of master hostname causes tserver crash

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1681: DNS resolution failure of master hostname causes tserver crash .. KUDU-1681: DNS resolution failure of master hostname causes tserver crash

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/4648/2/src/kudu/util/pstack_watcher.cc File

[kudu-CR] KUDU-1681: DNS resolution failure of master hostname causes tserver crash

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1681: DNS resolution failure of master hostname causes tserver crash .. Patch Set 2: (1 comment)

[kudu-CR] log: address a heap overflow race during log roll

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: log: address a heap overflow race during log roll .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4638 To unsubscribe, visit

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4648 to look at the new patch set (#2). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added

[kudu-CR] KUDU-1610. Bump trace viewer to new upstream version

2016-10-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1610. Bump trace viewer to new upstream version .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4642/1/thirdparty/vars.sh File

[kudu-CR] [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section

2016-10-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [web-ui] KUDU-1588 /tablets page should separate out tombstoned tablets to a separate section .. Patch Set 8: (1 comment)

[kudu-CR] [c++client/samples] added README for the sample

2016-10-06 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: [c++client/samples] added README for the sample .. Patch Set 3: (23 comments) Some style, wording, and grammar nits. Don't have the chance to test out the

[kudu-CR] [util] added Subprocess::GetExitStatus()

2016-10-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4648 Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subprocess::GetExitStatus() Added a method to collect exit