[parser] parser: sequence order option parse warning (pingcap#791)

you06 · Oct 9, 2021 · 2a7d4fa · 2a7d4fa
1 parent 02e3e4f
commit 2a7d4fa
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/parser/parser.go b/parser/parser.go
diff --git a/parser/parser.y b/parser/parser.y
@@ -11481,14 +11481,20 @@ SequenceOption:
 |	"ORDER"
 	{
 		$$ = &ast.SequenceOption{Tp: ast.SequenceOrder}
+		yylex.AppendError(yylex.Errorf("TiDB Sequence doesn't support ORDER option, ORDER will be parsed but ignored."))
+		parser.lastErrorAsWarn()
 	}
 |	"NOORDER"
 	{
 		$$ = &ast.SequenceOption{Tp: ast.SequenceNoOrder}
+		yylex.AppendError(yylex.Errorf("TiDB Sequence doesn't support ORDER option, NOORDER will be parsed but ignored."))
+		parser.lastErrorAsWarn()
 	}
 |	"NO" "ORDER"
 	{
 		$$ = &ast.SequenceOption{Tp: ast.SequenceNoOrder}
+		yylex.AppendError(yylex.Errorf("TiDB Sequence doesn't support ORDER option, NO ORDER will be parsed but ignored."))
+		parser.lastErrorAsWarn()
 	}
 
 SignedNum:
@@ -11515,7 +11521,7 @@ DropSequenceStmt:
 		}
 	}
 
-/********************************************************************
+/*******************************************************************
  * Index Advisor Statement
  *
  * INDEX ADVISE