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

use new sync.Map instead of handler container #358

Merged
merged 1 commit into from
Jul 20, 2022

Conversation

jkralik
Copy link
Member

@jkralik jkralik commented Jul 18, 2022

No description provided.

@jkralik jkralik force-pushed the jkralik/feature/removeHandlerContainer branch from b0349e4 to 2365547 Compare July 18, 2022 11:55
@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2022

Codecov Report

Merging #358 (1df4040) into v3 (3a905a5) will decrease coverage by 0.27%.
The diff coverage is 68.85%.

@@            Coverage Diff             @@
##               v3     #358      +/-   ##
==========================================
- Coverage   70.60%   70.32%   -0.28%     
==========================================
  Files          67       65       -2     
  Lines        5556     5480      -76     
==========================================
- Hits         3923     3854      -69     
- Misses       1237     1242       +5     
+ Partials      396      384      -12     
Impacted Files Coverage Δ
udp/discover.go 53.48% <50.00%> (-1.06%) ⬇️
udp/client/clientconn.go 72.34% <57.89%> (-0.03%) ⬇️
tcp/clientconn.go 67.82% <60.00%> (+1.26%) ⬆️
tcp/clientobserve.go 64.76% <71.42%> (-1.59%) ⬇️
udp/client/clientobserve.go 68.36% <71.42%> (-0.57%) ⬇️
tcp/session.go 70.77% <80.00%> (-0.08%) ⬇️
dtls/client.go 76.31% <100.00%> (ø)
dtls/server.go 84.00% <100.00%> (-1.72%) ⬇️
tcp/server.go 81.28% <100.00%> (ø)
udp/client.go 86.66% <100.00%> (ø)
... and 6 more

@jkralik jkralik force-pushed the jkralik/feature/createOnePoolMessage branch from dcf9cd2 to a46d636 Compare July 18, 2022 12:10
@jkralik jkralik force-pushed the jkralik/feature/removeHandlerContainer branch from 2365547 to f77d107 Compare July 18, 2022 12:11
Base automatically changed from jkralik/feature/createOnePoolMessage to v3 July 20, 2022 11:57
@jkralik jkralik force-pushed the jkralik/feature/removeHandlerContainer branch from f77d107 to 1df4040 Compare July 20, 2022 11:58
@jkralik jkralik merged commit 30deb51 into v3 Jul 20, 2022
@jkralik jkralik deleted the jkralik/feature/removeHandlerContainer branch July 20, 2022 12:15
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.

None yet

3 participants