Skip to content

Add client tests for psycopg2 python client#2231

Merged
fulghum merged 1 commit intomainfrom
fulghum/psycopg2
Jan 23, 2026
Merged

Add client tests for psycopg2 python client#2231
fulghum merged 1 commit intomainfrom
fulghum/psycopg2

Conversation

@fulghum
Copy link
Copy Markdown
Contributor

@fulghum fulghum commented Jan 22, 2026

Adds regression tests using psycopg2 for fix from #2216

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 22, 2026

Main PR
covering_index_scan_postgres 558.32/s 565.52/s +1.2%
index_join_postgres 120.59/s 119.98/s -0.6%
index_join_scan_postgres 181.78/s 181.05/s -0.5%
index_scan_postgres 11.39/s 11.08/s -2.8%
oltp_point_select 2347.93/s 2323.55/s -1.1%
oltp_read_only 1698.22/s 1696.77/s -0.1%
select_random_points 108.13/s 106.31/s -1.7%
select_random_ranges 434.65/s 430.20/s -1.1%
table_scan_postgres 10.99/s 11.00/s 0.0%
types_table_scan_postgres 5.11/s 4.99/s -2.4%

@github-actions
Copy link
Copy Markdown
Contributor

Main PR
Total 42090 42090
Successful 17565 17565
Failures 24525 24525
Partial Successes1 5583 5583
Main PR
Successful 41.7320% 41.7320%
Failures 58.2680% 58.2680%

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.

@fulghum fulghum requested a review from Hydrocharged January 23, 2026 06:09
Copy link
Copy Markdown
Collaborator

@Hydrocharged Hydrocharged left a comment

Choose a reason for hiding this comment

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

LGTM!

@fulghum fulghum merged commit 797357d into main Jan 23, 2026
17 checks passed
@fulghum fulghum deleted the fulghum/psycopg2 branch January 23, 2026 17:39
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