AfterShip / clickhouse-sql-parser / 30525248824

Pull #293

github

Leave a non-join GLOBAL/LOCAL where it was found

parseJoinLocality consumed the keyword before it knew a join followed, so
a locality with no join was swallowed: `SELECT * FROM t1 GLOBAL` parsed and
Format() returned `SELECT * FROM t1`. ClickHouse rejects that (Code 62), as
it does `FROM t global` — GLOBAL is not an implicit alias. Rewind to the
keyword instead, so the statement parser reports it.

The ARRAY JOIN error now points at the locality it names rather than at the
token the join op stopped on, "expected IN after GLOBAL" prints the token
instead of its kind, and parseJoinLocality drops a pos parameter that
parseJoinOp discards.

No golden fixture changes.

67 of 71 new or added lines in 3 files covered. (94.37%)

9132 of 16633 relevant lines covered (54.9%)

3156.02 hits per line