Adar Dembo has submitted this change and it was merged.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Fix kudu-ts-cli crash when there is no data in tablet
NULL pointer was being sent in as an
Dinesh Bhat has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 11:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4134/10/src/kudu/tools/kudu-ts-cli-test.cc
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 (#11).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Adar Dembo has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 10:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4134/10/src/kudu/tools/kudu-ts-cli-test.cc
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 10:
Build Started http://104.196.14.100/job/kudu-gerrit/3191/
--
To view, visit
Adar Dembo has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 9:
(6 comments)
Ugh, I forgot to publish the comments I had last night. I think Todd ended up
Dan Burkert has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 9:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4134/9/src/kudu/tools/kudu-ts-cli-test.cc
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 9:
Build Started http://104.196.14.100/job/kudu-gerrit/3188/
--
To view, visit
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 (#9).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Alexey Serbin has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 8:
(3 comments)
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 (#8).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 8:
Build Started http://104.196.14.100/job/kudu-gerrit/3185/
--
To view, visit
Todd Lipcon has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 7:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/4134/7/src/kudu/tools/kudu-ts-cli-test.cc
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 7: -Verified
Build Started http://104.196.14.100/job/kudu-gerrit/3180/
--
To view, visit
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 7:
Build Started http://104.196.14.100/job/kudu-gerrit/3161/
--
To view, visit
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 (#7).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Dan Burkert has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6: Code-Review+1
Code LGTM, but I agree that a test would be nice.
--
To view, visit
Todd Lipcon has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6:
Subprocess::Call has optional out-parameters for stdout/stderr so you can
capture it and
Dinesh Bhat has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6:
> Code change looks good, but how about a test in kudu-ts-cli-test?
Yess, thought about
Todd Lipcon has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6:
Code change looks good, but how about a test in kudu-ts-cli-test?
--
To view, visit
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6:
Build Started http://104.196.14.100/job/kudu-gerrit/3131/
--
To view, visit
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 (#6).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Dinesh Bhat has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4134/2/src/kudu/tools/ts-cli.cc
File
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 (#5).
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 5:
Build Started http://104.196.14.100/job/kudu-gerrit/3130/
--
To view, visit
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.
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
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 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
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
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
Kudu Jenkins has posted comments on this change.
Change subject: Fix kudu-ts-cli crash when there is no data in tablet
..
Patch Set 1:
Build Started http://104.196.14.100/job/kudu-gerrit/3090/
--
To view, visit
32 matches
Mail list logo