Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fix race when persisting create event #4404

Merged
merged 2 commits into from
Jan 18, 2019
Merged

Conversation

erikjohnston
Copy link
Member

Fix race when persisting create event …
When persisting a chunk of DAG it is sometimes requried to do a state
resolution, which requires knowledge of the room version. If this
happens while we're persisting the create event then we need to use that
event rather than attempting to look it up in the database.

When persisting a chunk of DAG it is sometimes requried to do a state
resolution, which requires knowledge of the room version. If this
happens while we're persisting the create event then we need to use that
event rather than attempting to look it up in the database.
@erikjohnston erikjohnston requested a review from a team January 16, 2019 14:59
@codecov-io
Copy link

Codecov Report

Merging #4404 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #4404   +/-   ##
========================================
  Coverage    73.65%   73.65%           
========================================
  Files          300      300           
  Lines        29815    29815           
  Branches      4897     4897           
========================================
  Hits         21961    21961           
  Misses        6412     6412           
  Partials      1442     1442

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f1a6a4...5520f3a. Read the comment docs.

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

lgtm

@hawkowl hawkowl merged commit 25dd56a into develop Jan 18, 2019
@hawkowl hawkowl deleted the erikj/create_version_race branch January 18, 2019 12:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants