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

sql: accept+ignore precision 6 on TIME,TIMESTAMP,TIMESTAMPTZ #35128

Merged
merged 1 commit into from
Feb 21, 2019

Conversation

knz
Copy link
Contributor

@knz knz commented Feb 21, 2019

Informs #32098.
Informs #32565.
This is not the full feature but might unlock Flowable compat. cc @rolandcrosby @awoods187 @drewdeally

Release note (sql change): CockroachDB now recognizes the syntax
TIME(6), TIMESTAMP(6) and TIMESTAMPTZ(6) / TIMESTAMP(6) WITH TIME ZONE, for compatibility with PostgreSQL. Only the value 6 is
supported (which is also the default in PostgreSQL). When used for a
table column definition, the precision is not stored and it is not
possible to distinguish types with and without specified precisions in
the introspection metadata.

Release note (sql change): CockroachDB now recognizes the syntax
`TIME(6)`, `TIMESTAMP(6)` and `TIMESTAMPTZ(6)` / `TIMESTAMP(6) WITH
TIME ZONE`, for compatibility with PostgreSQL. Only the value 6 is
supported (which is also the default in PostgreSQL). When used for a
table column definition, the precision is not stored and it is not
possible to distinguish types with and without specified precisions in
the introspection metadata.
@knz knz requested review from maddyblue and bobvawter February 21, 2019 18:43
@knz knz requested review from a team February 21, 2019 18:43
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz
Copy link
Contributor Author

knz commented Feb 21, 2019

bors r+

craig bot pushed a commit that referenced this pull request Feb 21, 2019
35128: sql: accept+ignore precision 6 on TIME,TIMESTAMP,TIMESTAMPTZ r=knz a=knz

Informs #32098.
Informs #32565.
This is not the full feature but might unlock Flowable compat. cc @rolandcrosby @awoods187  @drewdeally 

Release note (sql change): CockroachDB now recognizes the syntax
`TIME(6)`, `TIMESTAMP(6)` and `TIMESTAMPTZ(6)` / `TIMESTAMP(6) WITH
TIME ZONE`, for compatibility with PostgreSQL. Only the value 6 is
supported (which is also the default in PostgreSQL). When used for a
table column definition, the precision is not stored and it is not
possible to distinguish types with and without specified precisions in
the introspection metadata.

Co-authored-by: Raphael 'kena' Poss <knz@cockroachlabs.com>
@craig
Copy link
Contributor

craig bot commented Feb 21, 2019

Build succeeded

@craig craig bot merged commit c940821 into cockroachdb:master Feb 21, 2019
@knz knz deleted the 20190221-ts-prec branch February 22, 2019 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants