From: Kevin Lewis
Date: November 9 2008 6:03am
Subject: Re: Review fix for Bug#39696, serial log corruption
List-Archive: http://lists.mysql.com/falcon/163
Message-Id: <49167D1B.1010201@sun.com>
MIME-Version: 1.0
Content-Type: text/plain; format=flowed; charset=ISO-8859-1
Content-Transfer-Encoding: 7BIT

Chris,

Good work here.

Shouldn't this comment read;

-	// Flush record data, if any, and force the creation of a new serial 
log window
+	// Flush record data, if any, and force the creation of a new serial 
log record

What you have here is a fix for chilling records that avoids the problem 
of flushing a zero length record by not doing it.  But if a zero length 
serial log record is flushed anywhere else, will the reading of it 
become one byte off and cause an assert?  In that case, there are 
probably other related bugs.

Kevin

Christopher Powers wrote:
> Kevin,
> 
> Can you have a look at this commit: http://lists.mysql.com/commits/58261 ?
> 
> The falcon_chill_thaw stress test now completes on Xeno when it has 
> never done so before. I've pushed the fix to falcon-chris to see how it 
> does on Pushbuild2.
> 
> SerialLog::flush() bears a bit more scrutiny, and should be reviewed to 
> see how it handles zero-byte flushes.
> 
> Chris
>