Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Make returned collections respect explicit transactions #2385

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

AndrewSisley
Copy link
Contributor

Relevant issue(s)

Resolves #2384

Description

Makes returned collections respect explicit transactions.

I have not added any new tests in this PR as there is not much time before the release, there is currently no test actions to do facilitate this, and it is a fairly safe change. I might add tests in a new PR before 0.10.0 is cut if there is time.

@AndrewSisley AndrewSisley added bug Something isn't working area/collections Related to the collections system labels Mar 6, 2024
@AndrewSisley AndrewSisley added this to the DefraDB v0.10 milestone Mar 6, 2024
@AndrewSisley AndrewSisley requested a review from a team March 6, 2024 22:18
@AndrewSisley AndrewSisley self-assigned this Mar 6, 2024
Copy link
Collaborator

@fredcarle fredcarle left a comment

Choose a reason for hiding this comment

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

LGTM!

@AndrewSisley AndrewSisley merged commit b377876 into sourcenetwork:develop Mar 6, 2024
28 of 29 checks passed
@AndrewSisley AndrewSisley deleted the 2384-col-txn branch March 6, 2024 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/collections Related to the collections system bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collections returned from db.getCollections don't have a loaded transaction
2 participants