iotdb icon indicating copy to clipboard operation
iotdb copied to clipboard

Optimize the serialize method of intermedia result of aggregation

Open Beyyes opened this issue 1 year ago • 2 comments

Description

Content1 ...

Content2 ...

Content3 ...


This PR has:

  • [ ] been self-reviewed.
    • [ ] concurrent read
    • [ ] concurrent write
    • [ ] concurrent read and write
  • [ ] added documentation for new or modified features or behaviors.
  • [ ] added Javadocs for most classes and all non-trivial methods.
  • [ ] added or updated version, license, or notice information
  • [ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • [ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage.
  • [ ] added integration tests.
  • [ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Beyyes avatar Oct 22 '24 13:10 Beyyes

Codecov Report

Attention: Patch coverage is 0% with 88 lines in your changes missing coverage. Please review.

Project coverage is 39.84%. Comparing base (8bf2294) to head (e96f0af). Report is 16 commits behind head on master.

Files with missing lines Patch % Lines
.../operator/source/relational/aggregation/Utils.java 0.00% 41 Missing :warning:
...eryengine/plan/planner/TableOperatorGenerator.java 0.00% 15 Missing :warning:
...onal/aggregation/TableMaxMinByBaseAccumulator.java 0.00% 13 Missing :warning:
.../source/relational/aggregation/AvgAccumulator.java 0.00% 6 Missing :warning:
...ource/relational/aggregation/FirstAccumulator.java 0.00% 4 Missing :warning:
.../relational/TableAggregationTableScanOperator.java 0.00% 3 Missing :warning:
...rce/relational/aggregation/FirstByAccumulator.java 0.00% 2 Missing :warning:
...source/relational/aggregation/LastAccumulator.java 0.00% 2 Missing :warning:
...urce/relational/aggregation/LastByAccumulator.java 0.00% 2 Missing :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #13875      +/-   ##
============================================
+ Coverage     39.83%   39.84%   +0.01%     
  Complexity       71       71              
============================================
  Files          4153     4154       +1     
  Lines        262846   262839       -7     
  Branches      32005    32009       +4     
============================================
+ Hits         104692   104720      +28     
+ Misses       158154   158119      -35     

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

codecov[bot] avatar Oct 22 '24 14:10 codecov[bot]