Skip to content

Fix pipe tree database creation on receiver#17986

Merged
Caideyipi merged 1 commit into
apache:masterfrom
Caideyipi:fix-pipe-tree-database-auto-create
Jun 18, 2026
Merged

Fix pipe tree database creation on receiver#17986
Caideyipi merged 1 commit into
apache:masterfrom
Caideyipi:fix-pipe-tree-database-auto-create

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

This PR fixes tree-model pure-data pipe synchronization when the receiver does not already have the source tree database.

For pipe-generated tree inserts and TsFile loads, the receiver may carry an explicit tree database hint such as
oot.test.sg_0. Partition analysis uses this explicit database name directly, so the normal tree device-path based database auto-creation path is bypassed. As a result, receiver-side writes can repeatedly fail with Create DataPartition failed because the database ... is not exists.

The PR:

  • auto-creates explicit tree database hints on the DataNode receiver before executing tree insert/load statements
  • recalculates database-level when rebuilding pipe tree load retry statements
  • preserves the tree database hint when LoadTsFile fallback conversion replays tablets
  • adds a unit test for retry tree load statement database-level reconstruction

Verification

  • mvn -DskipTests spotless:apply -pl iotdb-core/datanode
  • git diff --check

mvn '-Ddevelocity.off=true' compile -pl iotdb-core/datanode was also attempted, but the current workspace fails in unrelated existing datanode/generated sources, e.g. missing Accumulator, IFill, LinearFill, PipeProcessorSubtask#getDisplayTaskID(), and related symbols.

@Caideyipi Caideyipi force-pushed the fix-pipe-tree-database-auto-create branch from 833048f to 52cf2da Compare June 18, 2026 09:23
@Caideyipi Caideyipi merged commit 52cf2da into apache:master Jun 18, 2026
4 of 40 checks passed
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