Skip to content

Fix quoted tree path parsing in table views - #18884

Merged
jt2594838 merged 1 commit into
apache:masterfrom
Caideyipi:fix/quoted-tree-view-prefix-path
Oct 15, 2026
Merged

jt2594838 merged 1 commit into
apache:masterfrom
Caideyipi:fix/quoted-tree-view-prefix-path

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Problem

A table-model view whose tree prefix path contains a space fails when the path is written with quoted identifiers:

AS root."`tv143_cmp_3`"."`a b`".**

The parser produced root.tv143_cmp_3.``a b``.**, which is rejected as an illegal path.

Cause

The relational parser already preserves backticks returned by parseNodeString for identifiers that require them, but parsePrefixPath wrapped every node in another pair of backticks.

Fix

Preserve an existing backtick-wrapped node name and only add backticks when the parsed node name is not already quoted. Add parser and integration regressions covering the tree-to-table view query.

Tests

  • mvn spotless:apply -pl iotdb-core/datanode
  • mvn spotless:apply -pl integration-test -P with-integration-tests
  • mvn compile -pl iotdb-core/datanode -am -DskipTests
  • CreateViewPrefixPathTest (1 test, 0 failures)
  • IoTDBTableIT#testCreateViewFromTreePathWithSpaces was attempted; the local integration environment could not start the ConfigNode and refused the connection.

@jt2594838
jt2594838 merged commit 673434b into apache:master Oct 15, 2026
42 of 43 checks passed
@jt2594838
jt2594838 deleted the fix/quoted-tree-view-prefix-path branch October 15, 2026 07:28
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.

2 participants