Skip to content
Snippets Groups Projects
  • Andrei Eres's avatar
    Send PeerViewChange with high priority (#4755) · 975e04bb
    Andrei Eres authored
    Closes https://github.com/paritytech/polkadot-sdk/issues/577
    
    ### Changed
    - `orchestra` updated to 0.4.0
    - `PeerViewChange` sent with high priority and should be processed first
    in a queue.
    - To count them in tests added tracker to TestSender and TestOverseer.
    It acts more like a smoke test though.
    
    ### Testing on Versi
    
    The changes were tested on Versi with two objectives:
    1. Make sure the node functionality does not change.
    2. See how the changes affect performance.
    
    Test setup:
    - 2.5 hours for each case
    - 100 validators
    - 50 parachains
    - validatorsPerCore = 2
    - neededApprovals = 100
    - nDelayTranches = 89
    - relayVrfModuloSamples = 50
    
    During the test period, all nodes ran without any crashes, which
    satisfies the first objective.
    
    To estimate the change in performance we used ToF charts. The graphs
    show that there are no spikes in the top as before. This proves that our
    hypothesis is correct.
    
    ### Normalized charts with ToF
    
    ![image](https://github.com/user-attachments/assets/0d49d0db-8302-4a8c-a557-501856805ff5)
    [Before](https://grafana.teleport.parity.io/goto/ZoR53ClSg?orgId=1)
    
    
    ![image](https://github.com/user-attachments/assets/9cc73784-7e45-49d9-8212-152373c05880)
    [After](https://grafana.teleport.parity.io/goto/6ux5qC_IR?orgId=1)
    
    ### Conclusion
    
    The prioritization of subsystem messages reduces the ToF of the
    networking subsystem, which helps faster propagation of gossip messages.
    Unverified
    975e04bb
Code owners
Assign users and groups as approvers for specific file changes. Learn more.
This project manages its dependencies using Cargo. Learn more