Skip to content

Comments

Start 2pc only on remote writes#5752

Open
markhannum wants to merge 1 commit intobloomberg:mainfrom
markhannum:disttxn_tests
Open

Start 2pc only on remote writes#5752
markhannum wants to merge 1 commit intobloomberg:mainfrom
markhannum:disttxn_tests

Conversation

@markhannum
Copy link
Contributor

Additionally the "no_coordinator_writes" test verifies that the coordinator master correctly acts as the coordinator even though there are no writes to the coordinator master itself. The "verify_no_distcommit" test verifies that we do not execute a distributed transaction if there are no foreign writes.

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
sc_truncate
reco-ddlk-sql
consumer_non_atomic_default_consumer_generated
unifiedcancel
sc_transactional_rowlocks_generated
writes_remsql_names_rte_connect_generated
writes_remsql_names_partial_index_off_generated
writes_remsql_names_fdbpushredirect_generated
writes_remsql_names_negativerowid_generated
writes_remsql_names_fdbpush_generated

Signed-off-by: Mark Hannum <mhannum@bloomberg.net>
Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
sp_snapshot_generated
consumer_non_atomic_default_consumer_generated
reco-ddlk-sql

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