Ability to modify SQL keyword list

[mustafaakin]

I know keywords come from the spec, however the popular words like domain, user, public is not actually useful in this library, not used in any way but we need to escape them as SELECT * FROM "user" which does not look nice. Would you consider the ability to customize that list as an advanced, somewhat hidden feature?

partiql-lang-kotlin/lang/src/org/partiql/lang/syntax/LexerConstants.kt

Line 30 in e455cf4

@JvmField internal val SQL92_KEYWORDS = setOf(

Parser Error: at line 1, column 8: unexpected keyword found, KEYWORD : public

[johnedquinn]

Hi @mustafaakin, we discussed this as a team, and it's a feature request that we are certainly interested in pursuing in the future. I can't say exactly when we'd implement this, but we actually have discussed to a certain degree how we can implement this with the upcoming PartiQLParser, a new implementation of Parser -- see #711. Linking the PR because it would subtantially decrease the time to roll this feature out.

SQL has reserved and non-reserved keywords, and I believe domain, user, and public are all non-reserved. With the new PartiQLParser, in #711, it is quite easy to add a rule in PartiQL.g4, perhaps called nonReservedKeyword which would return a PartiqlAst identifier.

[johnedquinn]

Update: PostgreSQL has a list of reserved vs non-reserved keywords on their website.

According to their implementation, user and public are reserved. We'd need to do research to see if it's at all possible to add user and public to PartiQL's future non-reserved keyword list.

[mustafaakin]

For now, we've forked LexerConstants.kt in the runtime, which works for us. For Postgres, I've tried the following, only user needs to be quoted, though, but it parses the query, but user resolves to the actual SQL user.

CREATE TABLE tmp3 (
    "user" TEXT,
    public BOOLEAN,
    domain TEXT
);

INSERT INTO tmp3 VALUES('mustafa', true, 'resmo.com');

SELECT * FROM tmp3 WHERE user = 'mustafa'; <-- does not match because SQL user is resmo
SELECT * FROM tmp3 WHERE "user" = 'mustafa'; 
SELECT * FROM tmp3 WHERE public AND domain = 'resmo.com'

[johnedquinn]

BTW @mustafaakin -- I created the above PR to add this to the new default parser (PartiQLParser). Should be extremely easy to modify the list of non-reserved keywords once it's merged.

[cemkucuk]

Hi @johnedquinn Is there any update on this?