Skip to content

IGNITE-26495 .NET: Fix TestReadOnlyTxSeesOldDataAfterUpdate flakiness#6706

Merged
ptupitsyn merged 6 commits intoapache:mainfrom
ptupitsyn:ignite-26495
Oct 6, 2025
Merged

IGNITE-26495 .NET: Fix TestReadOnlyTxSeesOldDataAfterUpdate flakiness#6706
ptupitsyn merged 6 commits intoapache:mainfrom
ptupitsyn:ignite-26495

Conversation

@ptupitsyn
Copy link
Copy Markdown
Contributor

No description provided.

@ptupitsyn ptupitsyn requested a review from Copilot October 6, 2025 13:51
@ptupitsyn ptupitsyn self-assigned this Oct 6, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the flakiness in the TestReadOnlyTxSeesOldDataAfterUpdate test by implementing a workaround for causality issues. The main change involves using a single-node connection instead of the default round-robin connection strategy to avoid observable timestamp issues with implicit client read-only transactions.

Key changes:

  • Added a workaround to use single-node connection configuration
  • Replaced shared client instance with local client instance using the modified configuration
  • Updated all database operations to use the new local client and record view

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ptupitsyn ptupitsyn requested a review from isapego October 6, 2025 14:29
@ptupitsyn ptupitsyn merged commit e06df80 into apache:main Oct 6, 2025
1 check passed
@ptupitsyn ptupitsyn deleted the ignite-26495 branch October 6, 2025 15:31
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