Skip to content

[no-release-notes] Add a test for https://github.com/dolthub/doltgresql/issues/1359#2140

Merged
fulghum merged 1 commit intomainfrom
fulghum/tests
Jan 6, 2026
Merged

[no-release-notes] Add a test for https://github.com/dolthub/doltgresql/issues/1359#2140
fulghum merged 1 commit intomainfrom
fulghum/tests

Conversation

@fulghum
Copy link
Copy Markdown
Contributor

@fulghum fulghum commented Dec 22, 2025

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 22, 2025

Main PR
covering_index_scan_postgres 543.16/s 554.78/s +2.1%
index_join_postgres 110.63/s 118.85/s +7.4%
index_join_scan_postgres 173.71/s 178.71/s +2.8%
index_scan_postgres 11.05/s 11.29/s +2.1%
oltp_point_select 2256.08/s 2226.21/s -1.4%
oltp_read_only 1703.29/s 1621.41/s -4.9%
select_random_points 107.45/s 101.94/s -5.2%
select_random_ranges 382.04/s 414.52/s +8.5%
table_scan_postgres 10.73/s 10.71/s -0.2%
types_table_scan_postgres 4.81/s 5.02/s +4.3%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 22, 2025

Main PR
Total 42090 42090
Successful 17505 17505
Failures 24585 24585
Partial Successes1 5587 5587
Main PR
Successful 41.5895% 41.5895%
Failures 58.4105% 58.4105%

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
Contributor

@codeaucafe codeaucafe left a comment

Choose a reason for hiding this comment

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

LGTM: pulled locally and ran test successfully. Thanks for adding this test based on findings in #1359

@fulghum fulghum requested a review from zachmu December 30, 2025 00:59
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

Comment thread testing/go/alter_table_test.go
@fulghum fulghum merged commit cebe501 into main Jan 6, 2026
17 checks passed
@fulghum fulghum deleted the fulghum/tests branch January 6, 2026 19:49
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