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

Avoid rebuilding Edu objects in worker mode #4770

Merged
merged 1 commit into from
Mar 4, 2019

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Feb 28, 2019

In worker mode, on the federation sender, when we receive an edu for sending over the replication socket, it is parsed into an Edu object. There is no point extracting the contents of it so that we can then immediately build another Edu.

@richvdh richvdh requested a review from a team February 28, 2019 21:01
@codecov
Copy link

codecov bot commented Feb 28, 2019

Codecov Report

Merging #4770 into develop will decrease coverage by 0.11%.
The diff coverage is 31.57%.

@@             Coverage Diff             @@
##           develop    #4770      +/-   ##
===========================================
- Coverage    75.13%   75.02%   -0.12%     
===========================================
  Files          340      340              
  Lines        34918    35000      +82     
  Branches      5722     5748      +26     
===========================================
+ Hits         26237    26260      +23     
- Misses        7067     7123      +56     
- Partials      1614     1617       +3

In worker mode, on the federation sender, when we receive an edu for sending
over the replication socket, it is parsed into an Edu object. There is no point
extracting the contents of it so that we can then immediately build another Edu.
Copy link
Member

@erikjohnston erikjohnston left a comment

Choose a reason for hiding this comment

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

I've run the sytest locally in worker mode and it passes :)

@richvdh
Copy link
Member Author

richvdh commented Mar 4, 2019

\o/

@richvdh richvdh merged commit 856c83f into develop Mar 4, 2019
@richvdh richvdh deleted the rav/optimise_edu_replication branch March 4, 2019 12:57
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.

2 participants