[PHOENIX-3263] Allow comma before CONSTRAINT to be optional

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Assignee: Eric Lomore
• Reporter: James R. Taylor
• Created: 11/Sep/16 05:50
• Updated: 22/Sep/16 21:40
• Resolved: 22/Sep/16 21:40

Description

In Phoenix, the comma before the CONSTRAINT is optional (which matches Oracle). Can this be supported in Calcite Phoenix?

For example, this is ok in Phoenix:

CREATE TABLE T (
    K VARCHAR
    CONSTRAINT PK PRIMARY KEY (K));

as is this:

CREATE TABLE T (
    K VARCHAR,
    CONSTRAINT PK PRIMARY KEY (K));

If this is not feasible, we could require the comma and change the tests. This is leading to a lot of failures.