Skip to content

[To dev/1.3] [Load] Handle empty path nodes during database auto-creation (#18307)#18319

Open
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-18307-dev-1.3
Open

[To dev/1.3] [Load] Handle empty path nodes during database auto-creation (#18307)#18319
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-18307-dev-1.3

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Backport #18307 to dev/1.3, cherry-picked from commit 39e2ad0.

LOAD now rejects malformed device paths during database auto-creation, ignores invalid database names left by older versions, and uses node-aware prefix matching so names such as root.sg and root.sg1 are not confused.

Adaptations for dev/1.3:

  • The tree schema auto-creation implementation is still embedded in LoadTsFileAnalyzer, so the fix and regression coverage are applied there.
  • PartialPath rejects malformed PlainDeviceID values during construction on this branch, so the backport converts that IllegalPathException to LoadAnalyzeException at the same boundary.
  • Legacy invalid database detection follows this branch's parser behavior while preserving the original structured-prefix semantics.

Tests:

  • mvn -o -nsu -pl iotdb-core/datanode spotless:apply
  • mvn -o -nsu -pl iotdb-core/datanode -Dtest=LoadTsFileAnalyzerTest test

Both commands passed; the targeted test class ran 2 tests with 0 failures.

…18307)

* Reject illegal device paths during LOAD

* Remove duplicate ConfigNode validation

* Handle legacy invalid databases during LOAD
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