Skip to content

Fix count and show timeseries follow-up - #18050

Merged
JackieTien97 merged 1 commit into
rc/2.0.10from
to-rc/2010
Jun 29, 2026
Merged

Fix count and show timeseries follow-up#18050
JackieTien97 merged 1 commit into
rc/2.0.10from
to-rc/2010

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Summary

  • Backport Fix count and show timeseries follow-up #17804 to rc/2.0.10.
  • Resolve region scan/count timeseries conflicts, including logical view active count handling and internal database visibility checks.
  • Add compatibility for schema source region skipping on this branch.

Verification

  • mvn spotless:apply -pl iotdb-core/datanode,integration-test -P with-integration-tests
  • git diff --cached --check
  • Targeted datanode tests were attempted, but datanode compilation is currently blocked by existing generated-source/Thrift cache mismatches such as missing IFill/PipeSubscribeHeartbeatReq and parser/thrift API mismatches.

* Fix count and show timeseries follow-up
* Refine internal timeseries count permissions
* fix
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecovBot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 48.77384% with 188 lines in your changes missing coverage. Please review.
✅ Project coverage is 40.85%. Comparing base (8da40c6) to head (3c45f43).

Files with missing linesPatch %Lines
...db/db/queryengine/plan/analyze/AnalyzeVisitor.java38.82%52 Missing ⚠️
...iotdb/db/queryengine/common/TimeseriesContext.java51.54%47 Missing ⚠️
...tor/source/ActiveTimeSeriesRegionScanOperator.java0.00%47 Missing ⚠️
...an/relational/security/TreeAccessCheckVisitor.java83.07%11 Missing ⚠️
...r/plan/node/metadata/read/TimeSeriesCountNode.java50.00%7 Missing ⚠️
...b/queryengine/plan/planner/LogicalPlanVisitor.java0.00%6 Missing ⚠️
...ueryengine/plan/planner/OperatorTreeGenerator.java0.00%6 Missing ⚠️
...n/node/metadata/read/LevelTimeSeriesCountNode.java53.84%6 Missing ⚠️
...operator/schema/CountGroupByLevelScanOperator.java57.14%3 Missing ⚠️
...execution/operator/schema/SchemaCountOperator.java33.33%2 Missing ⚠️
... and 1 more
Additional details and impacted files
@@ Coverage Diff @@## rc/2.0.10 #18050 +/- ##
===============================================
+ Coverage 40.82% 40.85% +0.03% 
Complexity 2610 2610 ===============================================
Files 5185 5185 Lines 351013 351317 +304 Branches 44895 44953 +58 ===============================================
+ Hits 143286 143528 +242 - Misses 207727 207789 +62 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JackieTien97
JackieTien97 merged commit 425bde4 into rc/2.0.10Jun 29, 2026
43 of 44 checks passed
@JackieTien97
JackieTien97 deleted the to-rc/2010 branch June 29, 2026 05:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Caideyipi@JackieTien97