You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@heron.apache.org by GitBox <gi...@apache.org> on 2018/04/04 17:36:52 UTC

[GitHub] nwangtw commented on issue #2847: Replace CHECK() with CHECK_*() so that when a check is failing, more ���

nwangtw commented on issue #2847: Replace CHECK() with CHECK_*() so that when a check is failing, more …
URL: https://github.com/apache/incubator-heron/pull/2847#issuecomment-378683450
 
 
   Yeah. It is a good point.
   
   I was changing the safe ones only in this PR to be quick. For the CHECK_EQ() << "...." case I need to do some tests to verify the behavior of the library. Let me see if I can find some time to run the tests.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services