apache / iotdb

Apache IoTDB
https://iotdb.apache.org/
Apache License 2.0
5.63k stars 1.03k forks source link

Change LogicalOptimizer and DistributionOptimizer to singleton #14158

Closed JackieTien97 closed 1 week ago

JackieTien97 commented 1 week ago

Optimizers are designed to be stateless, so we can share them among different queries. All stateful fields should be saved in Context instead of optimizer itself.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 54.16667% with 11 lines in your changes missing coverage. Please review.

Project coverage is 39.62%. Comparing base (f8bcf2a) to head (512879e). Report is 10 commits behind head on master.

Files with missing lines Patch % Lines
...ine/plan/relational/planner/TableModelPlanner.java 0.00% 5 Missing :warning:
...memory/TableModelStatementMemorySourceVisitor.java 0.00% 3 Missing :warning:
.../apache/iotdb/db/queryengine/plan/Coordinator.java 75.00% 2 Missing :warning:
...eryengine/plan/planner/TableOperatorGenerator.java 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #14158 +/- ## ============================================ - Coverage 39.65% 39.62% -0.03% Complexity 71 71 ============================================ Files 4224 4223 -1 Lines 269197 269265 +68 Branches 32623 32641 +18 ============================================ - Hits 106738 106703 -35 - Misses 162459 162562 +103 ```

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


🚨 Try these New Features: