Skip to content

Add DuckDB parity checks for point-in-time SQL features - #5

Merged
pavanamthomas merged 7 commits into
mainfrom
feat/duckdb-parity
Aug 24, 2026
Merged

Add DuckDB parity checks for point-in-time SQL features#5
pavanamthomas merged 7 commits into
mainfrom
feat/duckdb-parity

Conversation

@pavanamthomas

@pavanamthomas pavanamthomas commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Addresses #4 by testing the same point-in-time object through two SQL engines and an independent non-SQL calculation.

Implemented:

  • deterministic SQLite/DuckDB point-in-time fixture;
  • post-cutoff 99999.0 sentinel that correct historical features must exclude;
  • independent Pandas calculation of count, spend, and recency;
  • cross-engine NULL/no-history checks;
  • ROW_NUMBER, RANK, and DENSE_RANK parity on a known tie fixture;
  • DuckDB optional dependency exercised in CI;
  • docs/cross_engine_parity.md, README, and ROADMAP updates.

Validation: the current PR head passes CI.

Boundaries retained:

  • event time is not ingest time;
  • PostgreSQL-specific semantics and plans are not tested;
  • time zones are not modelled;
  • result parity on a deterministic subset is not a claim of full database equivalence.

The point is not another SQL badge. It is whether one information-set definition survives a second SQL engine and an independent Pandas check.

@pavanamthomas
pavanamthomas marked this pull request as ready for review August 24, 2026 04:18
@pavanamthomas
pavanamthomas merged commit 9e303af into main Aug 24, 2026
1 check passed
@pavanamthomas
pavanamthomas deleted the feat/duckdb-parity branch August 28, 2026 11:08
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