Skip to content

Reduce DNSIncoming parsing overhead#975

Merged
bdraco merged 4 commits into
python-zeroconf:masterfrom
bdraco:small_incoming_parser_performance
Sep 14, 2021
Merged

Reduce DNSIncoming parsing overhead#975
bdraco merged 4 commits into
python-zeroconf:masterfrom
bdraco:small_incoming_parser_performance

Conversation

@bdraco
Copy link
Copy Markdown
Member

@bdraco bdraco commented Sep 14, 2021

  • Parsing incoming packets is the most expensive operation
    zeroconf performs on networks with high mDNS volume

Screen Shot 2021-09-13 at 7 02 20 PM

- Parsing incoming packets is the most expensive operation
  zeroconf performance on networks with high mDNS volume
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 14, 2021

Codecov Report

Merging #975 (4839ed3) into master (5f52438) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #975   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        21    -1     
  Lines         2498      2489    -9     
  Branches       411       411           
=========================================
- Hits          2498      2489    -9     
Impacted Files Coverage Δ
zeroconf/__init__.py 100.00% <ø> (ø)
zeroconf/_protocol.py 100.00% <100.00%> (ø)
zeroconf/_services/browser.py 100.00% <100.00%> (ø)
zeroconf/_services/info.py 100.00% <100.00%> (ø)

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 5f52438...4839ed3. Read the comment docs.

@bdraco bdraco marked this pull request as ready for review September 14, 2021 20:11
@bdraco bdraco merged commit 78f9cd5 into python-zeroconf:master Sep 14, 2021
@bdraco bdraco deleted the small_incoming_parser_performance branch September 14, 2021 20:11
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