|
From: | Vijay Bellur |
Subject: | Re: [Gluster-devel] Query regarding GF_ASSERT macro |
Date: | Wed, 08 Jan 2014 12:00:58 +0530 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
On 01/08/2014 11:47 AM, Pranith Kumar Karampuri wrote:
----- Original Message -----From: "Harshavardhana" <address@hidden> To: "Vijay Bellur" <address@hidden> Cc: "Xavier Hernandez" <address@hidden>, address@hidden, "Atin Mukherjee" <address@hidden> Sent: Wednesday, January 8, 2014 11:41:33 AM Subject: Re: [Gluster-devel] Query regarding GF_ASSERT macroI think we need to do the following: 1. Change GF_ASSERT to be a simple wrapper over assert() and get rid of the -DDEBUG dependency. 2. Add -DNDEBUG for GA releases. qa releases to be built with assert() enabled.This may pose problems as customers/users see different failures compared to what QE found.
Ideally, all assertion failures need to be caught in our own testing before GA. The chances of assertion failures cropping up post GA should be very low. If we see more failures in GA releases due to assert() being a NOOP, it normally indicates that more testing is needed pre GA :).
-Vijay
[Prev in Thread] | Current Thread | [Next in Thread] |