Merge lp:~mmcm/akiban-sql-parser/more-types into lp:~akiban-technologies/akiban-sql-parser/trunk
Status: | Merged |
---|---|
Approved by: | Nathan Williams |
Approved revision: | 299 |
Merged at revision: | 297 |
Proposed branch: | lp:~mmcm/akiban-sql-parser/more-types |
Merge into: | lp:~akiban-technologies/akiban-sql-parser/trunk |
Diff against target: |
284 lines (+118/-6) 2 files modified
src/main/java/com/akiban/sql/types/TypeId.java (+44/-0) src/main/javacc/SQLGrammar.jj (+74/-6) |
To merge this branch: | bzr merge lp:~mmcm/akiban-sql-parser/more-types |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Nathan Williams | Approve | ||
Review via email: mp+161005@code.launchpad.net |
Description of the change
Handle more data types that the server already has (mostly for MySQL compatibility).
This one of a trio of branches: the others are for the server and the dumper. The overall goal is to always dump something that can be parsed (didn't used to be true) and come reasonably close to dumping something that reflects what's in the AIS. (The exceptional cases are noted in another branch.)
Just parse the {TINY,MEDIUM,
Parse TEXT that many databases have like CLOB.
None of these need to be reserved words (some of them were before this branch), so there isn't any real harm here, it's just trading off a parse error in the declaration.
Simple enough.