search

uber / aresdb

A GPU-powered real-time analytics storage and query engine.
https://eng.uber.com/aresdb/
Apache License 2.0
3k stars 233 forks source link

fix channel sender memleak #288

Closed shz117 closed 4 years ago

shz117 commented 4 years ago

caused by forgotten sender go routine of result channel

codecov[bot] commented 4 years ago

Codecov Report

Merging #288 into master will increase coverage by 0.02%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
+ Coverage   71.71%   71.73%   +0.02%     
==========================================
  Files         166      166              
  Lines       23096    23099       +3     
==========================================
+ Hits        16564    16571       +7     
+ Misses       5241     5236       -5     
- Partials     1291     1292       +1
Impacted Files Coverage Δ
broker/query_plan_non_agg.go 81.25% <100%> (+0.35%) :arrow_up:
subscriber/common/job/driver.go 67.73% <0%> (+1.41%) :arrow_up:

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 8ff5876...0ff802c. Read the comment docs.