Skip to content

[no-release-notes] more fixing - #3130

Merged
jycor merged 3 commits into
mainfrom
james/test-same
Aug 18, 2026
Merged

[no-release-notes] more fixing#3130
jycor merged 3 commits into
mainfrom
james/test-same

Conversation

@jycor

@jycor jycor commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor
Main PR
Total 42090 42090
Successful 18937 18935
Failures 23153 23155
Partial Successes1 5447 5447
Main PR
Successful 44.9917% 44.9869%
Failures 55.0083% 55.0131%

${\color{red}Regressions (2)}$

random

QUERY:          (SELECT unique1 AS random
  FROM onek ORDER BY random() LIMIT 1)
INTERSECT
(SELECT unique1 AS random
  FROM onek ORDER BY random() LIMIT 1)
INTERSECT
(SELECT unique1 AS random
  FROM onek ORDER BY random() LIMIT 1);
RECEIVED ERROR: expected row count 0 but received 1

subselect

QUERY:          select count(*) from tenk1 t
where (exists(select 1 from tenk1 k where k.unique1 = t.unique2) or ten < 0);
RECEIVED ERROR: timeout during Receive

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.

@itoqa

itoqa Bot commented Aug 18, 2026

Copy link
Copy Markdown

Ito QA test results
Commit: ec4653a: 1 test case ran, 1 passed ✅.

Summary

Coverage focused on preserving existing runtime behavior after a documentation-only change, including successful package validation and a representative database text lookup. This was a narrow happy-path and compatibility check, with no broader user flows, edge cases, or adversarial behavior exercised.

Safe to merge — the covered behavior remains healthy and the change introduces no application-code impact or attributable regressions. The run is low risk, with no merge blockers identified.

Tests run by Ito

View full run

Result Severity Type Description
Ambient The documentation-only change did not alter the application. The server/types package built successfully, and a local SQL query returned the expected text.

Tip

Reply with @itoqa to send us feedback on this test run.

@jycor jycor changed the title testing with no-op change [no-release-notes] more fixing Aug 18, 2026
@jycor
jycor enabled auto-merge (squash) August 18, 2026 19:29
@jycor
jycor merged commit 212ec76 into main Aug 18, 2026
26 of 27 checks passed
@jycor
jycor deleted the james/test-same branch August 18, 2026 19:59
@coffeegoddd

coffeegoddd commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@jycor DOLT

read_tests from_latency_median to_latency_median is_faster
covering_index_scan_postgres 2.43 2.43 0
groupby_scan_postgres 75.82 75.82 0
index_join_postgres 2.22 2.22 0
index_join_scan_postgres 1.58 1.58 0
index_scan_postgres 484.44 484.44 0
oltp_point_select 0.36 0.37 0
oltp_read_only 6.32 6.32 0
select_random_points 0.7 0.7 0
select_random_ranges 1.01 1.01 0
table_scan_postgres 493.24 484.44 0
types_table_scan_postgres 1191.92 1213.57 0
write_tests from_latency_median to_latency_median is_faster
bulk_insert 0.001 0.001 0
oltp_delete_insert_postgres 6.67 6.67 0
oltp_insert 3.36 3.36 0
oltp_read_write 13.22 13.46 0
oltp_update_index 3.55 3.55 0
oltp_update_non_index 3.25 3.25 0
oltp_write_only 6.91 6.91 0
types_delete_insert_postgres 7.17 7.17 0

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