uber / aresdb

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

hll memory estimate in MB instead of Bytes #312

Open jshencode opened 5 years ago

jshencode commented 5 years ago

this drastically under estimate memory usage for hll query and cause out of memory situation during massive hll query hits, such as weekly hll contracts

codecov[bot] commented 5 years ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.89%. Comparing base (56108d0) to head (6b14771). Report is 47 commits behind head on master.

Files with missing lines Patch % Lines
query/aql_processor.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #312 +/- ## ======================================== Coverage 71.88% 71.89% ======================================== Files 166 166 Lines 23252 23367 +115 ======================================== + Hits 16714 16799 +85 - Misses 5246 5275 +29 - Partials 1292 1293 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

CLAassistant commented 5 days ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Jian Shen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.