maria-developers team mailing list archive
-
maria-developers team
-
Mailing list archive
-
Message #05307
Re: Rev 3497: MDEV-318 IF (NOT) EXIST clauses for ALTER TABLE (MWL #252)
Hi, Alexey!
On Mar 29, Alexey Botchkov wrote:
> Hi, Sergei.
>
> > remove your new opt_if_not_exists_ident rule, write it like
> > - normal_key_type opt_ident key_alg '(' key_list ')'
> > + normal_key_type opt_if_not_exists opt_ident key_alg '(' key_list ')'
>
> I don't think it's a good idea. The 'opt_if_not_exists_ident' actually
> has the body:
> opt_if_not_exists_ident:
> opt_if_not_exists opt_ident
> {
> LEX *lex= Lex;
> if (lex->check_exists && lex->sql_command != SQLCOM_ALTER_TABLE)
> {
> my_parse_error(ER(ER_SYNTAX_ERROR));
> MYSQL_YYABORT;
> }
> $$= $2;
> };
>
> And i'd need to copy this everywhere instead of opt_**_ident call.
Right. Okay then, keep opt_if_not_exists_ident.
Regards,
Sergei
References