Skip to content

docs(cayenne): v2.2 preserves timestamp units; the 2.2.x snapshot still says microseconds - #2189

Merged
lukekim merged 1 commit into
trunkfrom
fix/cayenne-2-2-timestamp-units
Sep 10, 2026
Merged

lukekim merged 1 commit into
trunkfrom
fix/cayenne-2-2-timestamp-units

Conversation

@claudespice

Copy link
Copy Markdown
Contributor

Summary

The version-2.2.x Cayenne page still documents unconditional timestamp normalization:

  • Fully Supported Types: "Timestamps (normalized to Microsecond precision)"
  • Automatically Converted Types: a Timestamp(Nanosecond/...) → Timestamp(Microsecond) row, "Precision normalized"

Cayenne stopped doing that in v2.2.0. spiceai/spiceai#13180 split the rewrite lists in two and dropped TimestampToMicrosecond from the creation rules:

static CAYENNE_CREATION_REWRITE_RULES: TypeRewriteRules = &[&Float16ToFloat32];

// superset, because a table keeps the types it was created with
pub static CAYENNE_TYPE_REWRITE_RULES: TypeRewriteRules =
    &[&Float16ToFloat32, &TimestampToMicrosecond];

TimestampToMicrosecond survives only in the write-path recognition list, so an existing microsecond table fed by a nanosecond source is not misread as a schema change — it is not applied when creating a table. A table created on v2.2.0 or v2.2.1 keeps its source unit and timezone.

This is the "current-release behavioral change lands only in website/docs/" case: #2186 fixed vNext and left the snapshot the change shipped in stale.

Version scoping

Snapshot Latest tag Creation rewrite Docs Action
vNext — preserves unit preserves unit (#2186) —
version-2.2.x v2.2.1 preserves unit normalized fixed here
version-2.1.x v2.1.5 normalizes unconditionally normalized correct, untouched
version-2.0.x, 1.11.x — normalizes unconditionally normalized correct, untouched

At v2.1.5 schema.rs still has the unconditional arm, which is why the older snapshots are right as they stand:

if let DataType::Timestamp(unit, tz) = data_type && !matches!(unit, TimeUnit::Microsecond) {
    return Some(DataType::Timestamp(TimeUnit::Microsecond, tz.clone()));
}

What changed

website/versioned_docs/version-2.2.x/components/data-accelerators/cayenne/index.md only — the two claims above, replaced with the vNext wording, including the "Tables created before v2.2.0" note (the boundary is correct for this snapshot as written, since the change shipped in v2.2.0). mode: file_create is a real mode at v2.2.1.

Source refs

Test plan

  • cd website && npm run build passes (#on_schema_change exists in the 2.2.x datasets.md)
  • Cross-page grep: grep -rln "normalized to Microsecond" website/ now returns only 2.1.x, 2.0.x and 1.11.x, all correct
  • Files updated: 1

…ll said microseconds

spiceai/spiceai#13180 removed TimestampToMicrosecond from Cayenne's creation
rewrite rules, so a table created on v2.2.0 or v2.2.1 keeps its source unit and
timezone. The version-2.2.x snapshot still listed timestamps as normalized to
microseconds and carried the Timestamp(Nanosecond) -> Timestamp(Microsecond)
conversion row.

Mirrors the vNext wording from #2186 into the snapshot the change actually
shipped in. version-2.1.x and earlier are left alone: those releases really do
normalize unconditionally.
@github-actions

github-actions Bot commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

✅ Pull with Spice Passed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ Has at least one of the required labels: area/blog, area/docs, area/cookbook, dependencies
  • ✅ No banned labels detected
  • ✅ Has at least one assignee: claudespice

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Pull with Spice Failed

Passing checks:

  • ✅ Title meets minimum length requirement (10 characters)
  • ✅ No banned labels detected

Failed checks:

  • ❌ Please select at least one of the required labels for this pull request: area/blog, area/docs, area/cookbook, dependencies
  • ❌ At least one assignee is required for this pull request.

Please address these issues and update your pull request.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 deployed to https://a4c903d8.spiceai-org-website.pages.dev

@lukekim
lukekim merged commit 59c3ab9 into trunk Sep 10, 2026
6 of 10 checks passed
@lukekim
lukekim deleted the fix/cayenne-2-2-timestamp-units branch September 10, 2026 15:07

This branch was successfully deployed

1 active deployment
preview — f57132e0 Deployed Sep 10, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants