Re: bk commit into 6.0 tree (mhansson:1.2609) BUG#35020

[Marc Alff]

Hi Martin

Ok to push in 6.0 only, with minor polishing:

In the file query_cache.test, look for the following comments:

"
# all sql_cache statements, except for the first select, are ignored.
"

"
# sql_no_cache can occur in any nested select to turn on cacheing for 
the whole
# expression and it will always override a sql_cache statement.
"

and change them to reflect the current behavior.

Separating 'select_option' and 'query_expression_option' in the grammar 
is a good fix.

Regards,
Marc


mhansson@stripped wrote:
> Below is the list of changes that have just been committed into a local
> 6.0 repository of mhansson.  When mhansson does a push these changes
> will be propagated to the main repository and, within 24 hours after the
> push, to the public repository.
> For information on how to access the public repository
> see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html
>
> ChangeSet@stripped, 2008-03-10 10:22:24+01:00, mhansson@riffraff.(none) +8 -0
>   Bug#35020: illegal sql_cache select syntax
>   
>   The select option flags SQL_CACHE and SQL_NO_CACHE were available
>   for use in subqueries, which they're not supposed to.
>   
>   Fixed by creating special parser rules for select options in
>   subqueries. 
>
>