Merge lp:~mmcm/akiban-sql-parser/fewer-reserved-words into lp:~akiban-technologies/akiban-sql-parser/trunk
Proposed by
Mike McMahon
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Nathan Williams | ||||
Approved revision: | 304 | ||||
Merged at revision: | 304 | ||||
Proposed branch: | lp:~mmcm/akiban-sql-parser/fewer-reserved-words | ||||
Merge into: | lp:~akiban-technologies/akiban-sql-parser/trunk | ||||
Diff against target: |
355 lines (+46/-46) 1 file modified
src/main/javacc/SQLGrammar.jj (+46/-46) |
||||
To merge this branch: | bzr merge lp:~mmcm/akiban-sql-parser/fewer-reserved-words | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Nathan Williams | Approve | ||
Review via email: mp+178440@code.launchpad.net |
Description of the change
Move some keywords from reserved to non-reserved.
The process used was:
* Take list of SQL92 reserved words.
* Intersect with list of SQL2003 non-reserved words.
* Each of those that occurs in the reservedKeyword() production, move to nonReservedKeyw
* Undo any move that causes a JavaCC warning.
To post a comment you must log in.
Sounds good to me.