Skip to content

go: sqle/cluster: controller.go: Make DROP DATABASE replication participate in dolt_cluster_ack_writes_timeout_secs. - #11314

Merged
reltuk merged 2 commits into
mainfrom
aaron/cluster-ack-timeout-drop-database
Jul 17, 2026
Merged

go: sqle/cluster: controller.go: Make DROP DATABASE replication participate in dolt_cluster_ack_writes_timeout_secs.#11314
reltuk merged 2 commits into
mainfrom
aaron/cluster-ack-timeout-drop-database

Conversation

@reltuk

@reltuk reltuk commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

…cipate in dolt_cluster_ack_writes_timeout_secs.
@coffeegoddd

coffeegoddd commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@reltuk DOLT

read_tests from_latency to_latency percent_change
covering_index_scan 2.3 2.3 0.0
groupby_scan 142.39 139.85 -1.78
index_join 1.93 1.93 0.0
index_join_scan 1.3 1.3 0.0
index_scan 219.36 215.44 -1.79
oltp_point_select 0.25 0.25 0.0
oltp_read_only 5.0 5.0 0.0
select_random_points 0.5 0.5 0.0
select_random_ranges 0.64 0.64 0.0
table_scan 196.89 196.89 0.0
types_table_scan 458.96 458.96 0.0
write_tests from_latency to_latency percent_change
oltp_delete_insert 6.09 6.09 0.0
oltp_insert 3.13 3.13 0.0
oltp_read_write 11.24 11.24 0.0
oltp_update_index 3.3 3.25 -1.52
oltp_update_non_index 2.97 2.97 0.0
oltp_write_only 6.21 6.21 0.0
types_delete_insert 6.79 6.79 0.0

@coffeegoddd

Copy link
Copy Markdown
Contributor

@reltuk DOLT

comparing_percentages
100.000000 to 100.000000
version result total
2474acc ok 5937471
version total_tests
2474acc 5937471
correctness_percentage
100.0

@coffeegoddd

coffeegoddd commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@reltuk DOLT

test_name from_latency_p95 to_latency_p95 percent_change
tpcc-scale-factor-1 45.79 45.79 0.0
test_name from_server_name from_server_version from_tps to_server_name to_server_version to_tps percent_change
tpcc-scale-factor-1 dolt 041116f 53.11 dolt ac1753a 52.55 -1.05

@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

driver "github.com/dolthub/dolt/go/libraries/doltcore/dtestutils/sql_server_driver"
)

// TestClusterDropDatabaseParticipatesInAckWritesTimeout asserts that DROP

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.

Comment is probably a bit chatty and overspecified

@coffeegoddd

Copy link
Copy Markdown
Contributor

@reltuk DOLT

comparing_percentages
100.000000 to 100.000000
version result total
ac1753a ok 5937471
version total_tests
ac1753a 5937471
correctness_percentage
100.0

@reltuk
reltuk merged commit adfa309 into main Jul 17, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants