Skip to content

Conversation

@cvng
Copy link
Contributor

@cvng cvng commented Jan 8, 2024

What kind of change does this PR introduce?

add missing interval fields from from https://www.postgresql.org/docs/16/datatype-datetime.html

What is the current behavior?

parser panics

What is the new behavior?

parser returns

Additional context

as in #106, "multi-word" interval fields are required, eg. this query:

CREATE TABLE IF NOT EXISTS distributors (len interval hour to second(3));

expect this tokens:

tokens.push(TokenProperty::from(Token::HourP));

tokens.push(TokenProperty::from(Token::To)); tokens.push(TokenProperty::from(Token::SecondP));
@psteinroe psteinroe merged commit ab3c4ea into supabase-community:main Jan 8, 2024
@cvng cvng deleted the feat/interval-fields branch January 8, 2024 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants