[CALCITE-2993] ParseException may be thrown for legal SQL queries due to incorrect "LOOKAHEAD(1)" hints

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.20.0
• Labels: pull-request-available
• Assignee: Hongze Zhang
• Reporter: Hongze Zhang
• Created: 12/Apr/19 09:49
• Updated: 24/Jun/19 15:01
• Resolved: 23/Apr/19 15:27

Description

In some cases, ParseException is thrown incorrectly:

Case 1

select lead(x) ignore from t -- "ignore" aliases "lead(x)"

Case 2

select *
  from t match_recognize
  (
     after match skip to last -- "last" is a identifier. This is the implicit form of "skip to last last"
    pattern (strt down+ up+)
    define
      down as down.price < PREV(down.price),
      up as up.price > prev(up.price)
  ) mr

Case 3

select json_object(key: value) from t -- "key"/"value" are both column names.

And the failure on Case 1 is introduced by the fix of CALCITE-2847, Case 2/3 are not.

Issue Links

• links to: GitHub Pull Request #1159