search

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

block int64 for binary transformations #329

Closed shz117 closed 4 years ago

shz117 commented 4 years ago

int64 type can not be used for binary transformations for now. currently query fail until c++ type binding phase, add validation logic to fail those in compilation time.

codecov[bot] commented 4 years ago

Codecov Report

Merging #329 into master will decrease coverage by 0.03%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #329      +/-   ##
==========================================
- Coverage   72.01%   71.98%   -0.04%     
==========================================
  Files         167      167              
  Lines       23611    23640      +29     
==========================================
+ Hits        17004    17017      +13     
- Misses       5304     5321      +17     
+ Partials     1303     1302       -1
Impacted Files Coverage Δ
broker/query_compiler.go 60.03% <100%> (+0.79%) :arrow_up:
query/aql_compiler.go 75% <100%> (+0.15%) :arrow_up:
controller/mutators/etcd/schema_mutator.go 61.84% <0%> (-1.74%) :arrow_down:
memstore/cuckoo_index.go 90.26% <0%> (-1.01%) :arrow_down:
utils/http.go 62.5% <0%> (ø) :arrow_up:
api/common/response.go 0% <0%> (ø) :arrow_up:
api/query_handler.go 73.68% <0%> (ø) :arrow_up:
datanode/client/query_client.go 66.66% <0%> (+0.43%) :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 617d4ec...2a0f174. Read the comment docs.