[ 
https://issues.apache.org/jira/browse/THRIFT-215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12652245#action_12652245
 ] 

Erik Frey commented on THRIFT-215:
----------------------------------

I always thought compiling with -DNDEBUG was standard practice for release 
builds.  But it's not necessary within the scope of this bug.

> bad assert() in Monitor.cpp
> ---------------------------
>
>                 Key: THRIFT-215
>                 URL: https://issues.apache.org/jira/browse/THRIFT-215
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (C++)
>         Environment: Detected in linux 2.6.18-6
>            Reporter: Erik Frey
>            Assignee: Erik Frey
>            Priority: Minor
>         Attachments: compile_lib_ndebug.patch, monitor_bad_assert.patch
>
>
> On certain kernels, waiting with a very long timeout (e.g. many minutes or 
> hours) returns just before the expected time, hitting the assert and abruptly 
> bringing down the whole process.
> I don't have enough information about affected platforms to style an ifdef -- 
> feel free to buttress the attached patch if you'd prefer it to be more 
> specific.
> Also attached for your consideration is a patch to compile thrift lib with 
> -DNDEBUG.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to