Skip to content

Unskipping tests#1417

Merged
fulghum merged 1 commit intomainfrom
fulghum/unskip
Apr 28, 2025
Merged

Unskipping tests#1417
fulghum merged 1 commit intomainfrom
fulghum/unskip

Conversation

@fulghum
Copy link
Copy Markdown
Contributor

@fulghum fulghum commented Apr 23, 2025

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 23, 2025

Main PR
covering_index_scan_postgres 346.10/s 353.95/s +2.2%
index_join_postgres 152.35/s 151.22/s -0.8%
index_join_scan_postgres 186.12/s 185.30/s -0.5%
index_scan_postgres 12.42/s 12.30/s -1.0%
oltp_point_select 2648.16/s 2584.12/s -2.5%
oltp_read_only 1853.90/s 1842.65/s -0.7%
select_random_points 116.79/s 117.27/s +0.4%
select_random_ranges 131.76/s 130.01/s -1.4%
table_scan_postgres 11.33/s 11.83/s +4.4%
types_table_scan_postgres 5.50/s 5.50/s 0.0%

@github-actions
Copy link
Copy Markdown
Contributor

Main PR
Total 42090 42090
Successful 16361 16361
Failures 25729 25729
Partial Successes1 5530 5530
Main PR
Successful 38.8715% 38.8715%
Failures 61.1285% 61.1285%

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.

Copy link
Copy Markdown
Member

@zachmu zachmu 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 0faea77 into main Apr 28, 2025
14 checks passed
@fulghum fulghum deleted the fulghum/unskip branch April 28, 2025 23:01
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.

3 participants