Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various parser bugs / missing features #7171

Closed
68 tasks done
lukaseder opened this issue Feb 15, 2018 · 8 comments
Closed
68 tasks done

Various parser bugs / missing features #7171

lukaseder opened this issue Feb 15, 2018 · 8 comments

Comments

@lukaseder
Copy link
Member

lukaseder commented Feb 15, 2018

This is a continuation of #6485 and will be continued in #7518


Some features were not yet supported in the jOOQ 3.9 parser. We'll fix them all in this single issue in order not to create too many issues. Here's the list of fixed things:

SELECT features

Other statements

Table expressions

Column expressions (and predicates)

Literals and data types

Other improvements

New features

  • Parse and fail clauses available only in the pro edition
@lukaseder lukaseder added this to the Version 3.11.0 milestone Feb 15, 2018
lukaseder added a commit that referenced this issue Mar 1, 2018
- [#7237] now() function
- [#7171] current_timestamp(), current_time(), and current_date() can be functions, not just literals
lukaseder added a commit that referenced this issue Mar 9, 2018
@mkmatis
Copy link

mkmatis commented Apr 12, 2018

Hi, I discovered that DDLDatabase fails when script contains CREATE INDEX and column name is the same as H2 data type, e.g.:

CREATE TABLE TASK_HISTORY
(
    TIMESTAMP TIMESTAMP NOT NULL
);

CREATE INDEX TASK_HISTORY_TIMESTAMP_INDEX ON TASK_HISTORY(TIMESTAMP);

The same thing happens for DATE.

@lukaseder
Copy link
Member Author

Interesting, thanks for reporting. The parser parses the TIMESTAMP keyword as the beginning of a TIMESTAMP literal. The workaround would be to quote the identifier:

CREATE INDEX TASK_HISTORY_TIMESTAMP_INDEX ON TASK_HISTORY("TIMESTAMP");

If H2 accepts the syntax, we'll have to, as well, of course

@lukaseder
Copy link
Member Author

Remaining tasks moved to #7518

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants