Skip to content

Strictly isolate pipe visibility and fix zstd compression IT#17917

Open
Caideyipi wants to merge 7 commits into
apache:masterfrom
Caideyipi:strict-pipe-tree-table-isolation
Open

Strictly isolate pipe visibility and fix zstd compression IT#17917
Caideyipi wants to merge 7 commits into
apache:masterfrom
Caideyipi:strict-pipe-tree-table-isolation

Conversation

@Caideyipi

@Caideyipi Caideyipi commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Make pipe management visibility follow the SQL dialect stored in source parameters instead of capture.tree/capture.table.
  • Keep capture semantics unchanged for cross-capture and double-living pipes.
  • Update isolation IT expectations and add unit coverage for dialect-based pipe visibility.
  • Fix IoTDBPipeSinkCompressionIT#testZstdCompressorLevel by asserting synced data after the valid zstd-level pipes start, avoiding a flaky count timeseries assertion on historical TsFile/schema visibility.

Tests

  • mvn -Ddevelocity.off=true -pl iotdb-core/node-commons -Dtest=VisibilityUtilsTest test
  • mvn -Ddevelocity.off=true -pl integration-test -P with-integration-tests spotless:check
  • mvn -pl integration-test -P with-integration-tests spotless:apply
  • git diff --check

Targeted IoTDBPipeSinkCompressionIT#testZstdCompressorLevel execution was attempted, but local verification was blocked before IT execution by unrelated datanode generated aggregation compile errors with -am; without -am, Maven could not resolve several remote 2.0.7-SNAPSHOT artifacts.

@Caideyipi Caideyipi changed the title Strictly isolate pipe tree and table visibility Strictly isolate pipe visibility and fix zstd compression IT Jun 12, 2026
Sign up for free to 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.

1 participant