Skip to content

[CALCITE-2299] TIMESTAMPADD(SQL_TSI_FRAC_SECOND) should be nanoseconds - #731

Closed
snuyanzin wants to merge 1 commit into
apache:masterfrom
snuyanzin:CALCITE_2299
Closed

[CALCITE-2299] TIMESTAMPADD(SQL_TSI_FRAC_SECOND) should be nanoseconds #731
snuyanzin wants to merge 1 commit into
apache:masterfrom
snuyanzin:CALCITE_2299

Conversation

@snuyanzin

Copy link
Copy Markdown
Contributor

Calcite part for [CALCITE-2299] (needs avatica updates for being compiled and working)

  1. Added support for nanoseconds
  2. Fix SQL_TSI_FRAC_SECOND which should be interpreted as nanoseconds.
  3. Fix not working case timestampadd(MICROSECOND, 2000000, timestamp '2016-02-24 12:42:25')
  4. Added tests

@snuyanzin
snuyanzin force-pushed the CALCITE_2299 branch 2 times, most recently from bb3f097 to 97b993c Compare July 2, 2018 11:18
1) Added support for nanoseconds
2) Fix SQL_TSI_FRAC_SECOND which should be interpreted as nanoseconds.
3) Added tests
@asfgit asfgit closed this in 3fa2945 Jul 9, 2018
wangxlong pushed a commit to wangxlong/calcite that referenced this pull request Feb 13, 2020
…s (Sergey Nuyanzin)

1) Add NANOSECONDS time unit;
2) Fix SQL_TSI_FRAC_SECOND which should be interpreted as nanoseconds;
3) Add tests.

Close apache#731
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Aug 28, 2025
…s (Sergey Nuyanzin)

1) Add NANOSECONDS time unit;
2) Fix SQL_TSI_FRAC_SECOND which should be interpreted as nanoseconds;
3) Add tests.

Close apache#731

Change-Id: I985c02ee6dcff61c04cea66092ee42747e6ae933
jamesstarr pushed a commit to jamesstarr/calcite that referenced this pull request Mar 16, 2026
…s (Sergey Nuyanzin)

1) Add NANOSECONDS time unit;
2) Fix SQL_TSI_FRAC_SECOND which should be interpreted as nanoseconds;
3) Add tests.

Close apache#731

Change-Id: I985c02ee6dcff61c04cea66092ee42747e6ae933
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.

1 participant