Skip to content

add nightly ORM test - #2291

Merged
jennifersp merged 5 commits into
mainfrom
jennifer/orm-tests
Feb 12, 2026
Merged

jennifersp merged 5 commits into
mainfrom
jennifer/orm-tests

Conversation

@jennifersp

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Feb 10, 2026 •

Copy link
Copy Markdown
Contributor
Main PR
covering_index_scan_postgres 559.97/s 543.85/s -2.9%
index_join_postgres 114.79/s 115.79/s +0.8%
index_join_scan_postgres 170.24/s 168.53/s -1.1%
index_scan_postgres 10.96/s 10.89/s -0.7%
oltp_point_select 2274.75/s 2173.93/s -4.5%
oltp_read_only 1653.89/s 1627.57/s -1.6%
select_random_points 102.69/s 101.60/s -1.1%
select_random_ranges 414.43/s 414.64/s 0.0%
table_scan_postgres 10.71/s 10.69/s -0.2%
types_table_scan_postgres 4.96/s 4.89/s -1.5%

@github-actions

github-actions Bot commented Feb 10, 2026 •

Copy link
Copy Markdown
Contributor
Main PR
Total 42090 42090
Successful 17559 17559
Failures 24531 24531
Partial Successes1 5585 5585
Main PR
Successful 41.7177% 41.7177%
Failures 58.2823% 58.2823%

Footnotes

  1. These are tests that we're marking as Successful, however they do not match the expected output in some way. This is due to small differences, such as different wording on the error messages, or the column names being incorrect while the data itself is correct. ↩

@jennifersp
jennifersp requested a review from zachmu February 11, 2026 17:21

@zachmu zachmu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the good test, but two things:

  1. This should go in testing/postgres-client-tests
  2. It should run on every PR, not nightly

Comment thread testing/postgres-client-tests/drizzle/src/index.ts
@jennifersp
jennifersp requested a review from zachmu February 11, 2026 20:40

@zachmu zachmu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks for the changes

Comment thread testing/postgres-client-tests/postgres-client-tests.bats
@jennifersp
jennifersp enabled auto-merge February 12, 2026 19:10
@jennifersp
jennifersp merged commit 675fc98 into main Feb 12, 2026
17 checks passed
@jennifersp
jennifersp deleted the jennifer/orm-tests branch February 12, 2026 20:04
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.

2 participants