search

winedepot / pinot

Apache Pinot (Incubating) - A realtime distributed OLAP datastore
https://pinot.apache.org
Apache License 2.0
1 stars 0 forks source link

Adding pinot query to broker request converter #16

Closed kishoreg closed 5 years ago

kishoreg commented 5 years ago

Part of #14 Initial implementation to convert. Will add test cases once we have the PQL -> PinotQuery compiler.

codecov-io commented 5 years ago

Codecov Report

Merging #16 into develop will decrease coverage by 0.09%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             develop      #16     +/-   ##
============================================
- Coverage      64.04%   63.95%   -0.1%     
  Complexity         4        4             
============================================
  Files           1044     1045      +1     
  Lines          53467    53591    +124     
  Branches        7600     7623     +23     
============================================
+ Hits           34245    34273     +28     
- Misses         16884    16982     +98     
+ Partials        2338     2336      -2
Impacted Files Coverage Δ Complexity Δ
...pql/parsers/PinotQuery2BrokerRequestConverter.java 0% <0%> (ø) 0 <0> (?)
...r/validation/RealtimeSegmentValidationManager.java 58.33% <0%> (-25.01%) 0% <0%> (ø)
...egation/function/customobject/MinMaxRangePair.java 75.86% <0%> (-24.14%) 0% <0%> (ø)
...impl/dictionary/DoubleOnHeapMutableDictionary.java 68.88% <0%> (-13.34%) 0% <0%> (ø)
...e/operator/dociditerators/SortedDocIdIterator.java 55.55% <0%> (-8.34%) 0% <0%> (ø)
...elix/core/periodictask/ControllerPeriodicTask.java 76.31% <0%> (-7.9%) 0% <0%> (ø)
...impl/dictionary/FloatOffHeapMutableDictionary.java 81.81% <0%> (-5.46%) 0% <0%> (ø)
...inot/transport/netty/NettyTCPClientConnection.java 81.45% <0%> (-5.3%) 0% <0%> (ø)
...a/org/apache/pinot/common/config/TagNameUtils.java 64.1% <0%> (-5.13%) 0% <0%> (ø)
...va/org/apache/pinot/controller/ControllerConf.java 46.7% <0%> (-1.65%) 0% <0%> (ø)
... and 26 more

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 9751e3c...8ae0eb5. Read the comment docs.