Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14908 )

Change subject: [master/tserver] return non-zero status from main() instead of 
crashing
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14908/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/14908/1//COMMIT_MSG@17
PS1, Line 17: In most cases, core dumps were disabled, and only minidumps were
            :     available from processes crashed in such cases.
> Corefiles don't help debug issues in Init/Start?
This statement says that minidumps (not core dumps) are not helpful in those 
scenarios because minidumps don't contain the heap data, and the information 
about the resturned Status (code, message, everything) is stored in the heap.

I'll take a look at the CM's behavior w.r.t. to this change, yup.  I don't 
think it will change much since any non-zero return code should have been 
considered as non-success return code anyways.



--
To view, visit http://gerrit.cloudera.org:8080/14908
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id06646e2211eb24db28c582455d4a34af7501b26
Gerrit-Change-Number: 14908
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 16 Dec 2019 20:30:37 +0000
Gerrit-HasComments: Yes

Reply via email to