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

2016-10-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subprocess::GetExitStatus() The Subprocess::GetExitStatus() method allows to retrive exit statu

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

2016-10-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4648 To unsubscribe, visit http://gerrit.

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

2016-10-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4648 To unsubscribe, visit http://gerrit.

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

2016-10-18 Thread Alexey Serbin (Code Review)
Hello Dinesh Bhat, 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 (#12). Change subject: [util] added Subprocess::GetExitStatus() .. [util]

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

2016-10-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 10: (6 comments) Thank you for the review! Will post new version in a moment. http://gerrit.cloudera.org

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

2016-10-18 Thread Alexey Serbin (Code Review)
Hello Dinesh Bhat, 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 (#11). Change subject: [util] added Subprocess::GetExitStatus() .. [util]

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

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

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

2016-10-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4648 To unsubscribe, visit http://gerrit

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

2016-10-12 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 (#10). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subpro

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

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

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

2016-10-12 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 9: (5 comments) http://gerrit.cloudera.org:8080/#/c/4648/7/src/kudu/util/subprocess.cc File src/kudu/util/su

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

2016-10-11 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 (#9). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subproc

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

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

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

2016-10-11 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 8: (5 comments) http://gerrit.cloudera.org:8080/#/c/4648/7/src/kudu/util/subprocess.cc File src/kudu/util/su

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

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

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

2016-10-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new patch set (#8). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subprocess::GetExitStatus() The Subprocess::GetExitStatus() method allows to retrive exit status and exit

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

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

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

2016-10-10 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 7: (6 comments) http://gerrit.cloudera.org:8080/#/c/4648/7/src/kudu/util/subprocess.cc File src/kudu/util/su

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

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

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

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

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

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

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

2016-10-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 7: Verified+1 The failures in Jenkins build are from bouncing between old thirdparty and new thirdparty si

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

2016-10-08 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 (#7). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subproc

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

2016-10-08 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 (#6). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subproc

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

2016-10-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/4648/5//COMMIT_MSG Commit Message: PS5, Line 13: The

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

2016-10-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [util] added Subprocess::GetExitStatus() .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/4648/5//COMMIT_MSG Commit Message: PS5, Line 13: The mo

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

2016-10-07 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 (#5). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subproc

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

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

[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 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4648/4/src/kudu/util/subprocess-test.cc File src/kudu/util

[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 Subproc

[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 src/kudu/

[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 src/kudu/u

[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 src/kudu/uti

[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 (#3). Change subject: [util] added Subprocess::GetExitStatus() .. [util] added Subproc

[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 src/kudu/u

[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 src/kudu/util

[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 Subproc

[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 stat