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

Ftr: change timeQ from limit chan to ultimate chan #77

Merged
merged 5 commits into from
Nov 18, 2021

Conversation

Lvnszn
Copy link
Contributor

@Lvnszn Lvnszn commented Nov 16, 2021

What this PR does:
change timeQ from limit chan to ultimate chan
Which issue(s) this PR fixes:

Fixes

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


@Lvnszn Lvnszn changed the title WIP:Ftr: change timeQ from limit chan to ultimate chan Ftr: change timeQ from limit chan to ultimate chan Nov 16, 2021
time/timer.go Outdated Show resolved Hide resolved
@LaurenceLiZhixin
Copy link
Contributor

@Lvnszn Please run go fmt.

@Lvnszn Lvnszn force-pushed the feature/unbound_timer_channel branch from 8b6841e to e46db36 Compare November 17, 2021 05:38
@Lvnszn
Copy link
Contributor Author

Lvnszn commented Nov 17, 2021

@Lvnszn Please run go fmt.

address

time/timer.go Outdated Show resolved Hide resolved
time/timer.go Outdated Show resolved Hide resolved
time/timer.go Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Merging #77 (e46db36) into master (9220bf2) will increase coverage by 0.04%.
The diff coverage is 72.72%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   73.27%   73.31%   +0.04%     
==========================================
  Files          45       45              
  Lines        4550     4542       -8     
==========================================
- Hits         3334     3330       -4     
+ Misses       1013     1008       -5     
- Partials      203      204       +1     
Impacted Files Coverage Δ
time/timer.go 78.52% <72.72%> (+0.70%) ⬆️
database/kv/zk/client.go 44.51% <0.00%> (-0.38%) ⬇️

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 9220bf2...e46db36. Read the comment docs.

@AlexStocks AlexStocks merged commit 7eddead into dubbogo:master Nov 18, 2021
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.

5 participants