Skip to content

Merge upstream#459

Merged
SergeiPavlov merged 92 commits intomaster-servicetitanfrom
mergeUpstream
Mar 20, 2026
Merged

Merge upstream#459
SergeiPavlov merged 92 commits intomaster-servicetitanfrom
mergeUpstream

Conversation

@SergeiPavlov
Copy link
Copy Markdown
Collaborator

No description provided.

If transaction has become broken during commit its rollback leads to
new exception which will overwrite the original one.
Check for active transaction should work because on exception within
driver.CommitTransaction() active transaction is set to NULL.
Async counterparts for QueryEndpoint.Single/SingleOrDefault
# Conflicts:
#	ChangeLog/7.2.1-dev.txt
also null check (??) applied if possible
also addes some missing validation
- is/is not null applied
- some "result is never used" warnings eliminated
- summaries to public extension methods added
- other things
…disposal

Addresses issue of not disposing IAsyncEnumerator which causes query being open
…te-on-commit-upgrade

No exception overwriting when it happens on upgrade transaction commit
- less enumerations
- less collections created
@SergeiPavlov SergeiPavlov requested a review from botinko as a code owner March 19, 2026 23:20
@SergeiPavlov SergeiPavlov removed the request for review from botinko March 19, 2026 23:21
@SergeiPavlov SergeiPavlov merged commit 8f93c60 into master-servicetitan Mar 20, 2026
6 checks passed
@SergeiPavlov SergeiPavlov deleted the mergeUpstream branch March 20, 2026 21:09
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