Skip to content

docs(sql): timestamp without time zone decoded as UTC - #40160

Closed
deepshekhardas wants to merge 1 commit into
oven-sh:mainfrom
deepshekhardas:fix-39879-postgres-timestamp-docs
Closed

docs(sql): timestamp without time zone decoded as UTC#40160
deepshekhardas wants to merge 1 commit into
oven-sh:mainfrom
deepshekhardas:fix-39879-postgres-timestamp-docs

Conversation

@deepshekhardas

Copy link
Copy Markdown

Fixes #39879

docs(sql): clarify that timestamp without time zone is decoded as UTC (PostgreSQL OID 1114).

…ven-sh#39879)

PostgreSQL timestamp (OID 1114) is decoded as UTC so text and binary paths agree and bound Date round-trips regardless of host timezone. This differs from node-postgres/postgres.js/PGlite which decode as local time. Add PostgreSQL-specific note with callout.

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Your included review limit has been reached.

You’re in a promotional period — use the checkbox below to run this review for free:

  • Run review for free

On-demand reviews are free for the next 29 days. After that, they cost $0.25 per reviewed file.

How can I continue?

Run this review now using the option above, or comment @coderabbitai review --use-credits.

You can also wait for the limit to reset (next review available in 46 minutes), then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a0f65103-b270-4172-a586-72809fcee228

📥 Commits

Reviewing files that changed from the base of the PR and between 8eb5b6e and 1c2f0f0.

📒 Files selected for processing (1)
  • docs/runtime/sql.mdx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@robobun

robobun commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

Thank you for the PR. This has the same diff as #39911, which was closed on August 21 in favor of #39881, and #39881 is still open for #39879. To keep one PR per issue in the review queue, I am closing this one as a duplicate of #39881.

The extra facts from #39911 (OID 1114, text and binary queries agree, PGlite) are already in #39881. Commit b187c78 there carries a Co-authored-by trailer with your name. That PR also adds a top-level "Dates and time zones" section that states the rule once for PostgreSQL and MySQL.

@robobun robobun closed this Aug 23, 2026
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.

docs: timestamp without time zone decoding as UTC isn't documented in the PostgreSQL section

2 participants