From: Michael Widenius
Date: March 8 2009 1:49am
Subject: re: bzr commit into MySQL/Maria:mysql-maria branch (guilhem:2736) Bug#42180
List-Archive: http://lists.mysql.com/maria/452
Message-Id: <18867.9259.892898.732908@narttu.askmonty.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit


Hi!

>>>>> "Guilhem" == Guilhem Bichot <guilhem@stripped> writes:

Guilhem> #At bzr+ssh://bk-internal.mysql.com/bzrroot/server/mysql-maria/ based on revid:guilhem@stripped
Guilhem>  2736 Guilhem Bichot	2009-03-05
Guilhem>       Fix for BUG#42180 "Maria: maria-recovery-bitmap.test fails repeatedly"
Guilhem>       modified:
Guilhem>         storage/maria/ma_loghandler.c
Guilhem>         storage/maria/ma_test_force_start.pl
Guilhem>         storage/maria/trnman.c

<cut>

-    if  ((undo_lsn= trn->undo_lsn) == 0) /* trn can be forgotten */
+    if ((LSN_WITH_FLAGS_TO_FLAGS(trn->first_undo_lsn) &
+         TRANSACTION_LOGGED_LONG_ID) == 0)
       continue;

Another option would be to just use:

+    if (!(trn->first_undo_lsn & TRANSACTION_LOGGED_LONG_ID))


Had no other comments on patch. Looked ok!

Can you please merge this to lp:maria also ?

Regards,
Monty