From: Jorgen Loland
Date: February 4 2011 9:29am
Subject: Re: bzr commit into mysql-5.1 branch (ole.john.aske:3573) Bug#59308
List-Archive: http://lists.mysql.com/commits/130374
Message-Id: <4D4BC6FE.8010509@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Hi Ole John

The patch is already approved, I just noticed this:

On 02/02/2011 03:47 PM, Ole John Aske wrote:
> @@ -13771,9 +13724,10 @@ check_reverse_order:
>   	Don't reverse the sort order, if it's already done.
>           (In some cases test_if_order_by_key() can be called multiple times
>         */
> -      if (!select->quick->reverse_sorted())
> +      if (select->quick->reverse_sorted())
> +        goto skiped_sort_order;

Good catch.

-- 
Jørgen Løland | Senior Software Engineer | +47 73842138
Oracle MySQL
Trondheim, Norway