You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@iotdb.apache.org by GitBox <gi...@apache.org> on 2022/11/09 02:49:20 UTC

[GitHub] [iotdb] Cpaulyz opened a new pull request, #7943: [IOTDB-4876] Fix PathPatternTree missing information on subpaths

Cpaulyz opened a new pull request, #7943:
URL: https://github.com/apache/iotdb/pull/7943

   ## Description
   
   ### BUG description
    After appending root.sg.d and root.sg.d.** to an empty pathPatternTree and invoking constructTree(), only root.sg.d.** can be taken by invoking getAllPathPatterns() and root.sg.d is missing.
   
   ### Solution
   In the original design, only the leaf nodes on the PathPatternTree are considered as measurement. However, the information is lost when there is a subpath for measurement. e.g. root->sg->d(measurement)->**(measurement).
   
   So we use an additional roleType field to identify whether the node in the tree is internal or measurement.
   
   To avoid compatibility problema, new semantics are given to the original valueSet field in the serialization method.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [iotdb] liuminghui233 merged pull request #7943: [IOTDB-4876] Fix PathPatternTree missing information on subpaths

Posted by GitBox <gi...@apache.org>.
liuminghui233 merged PR #7943:
URL: https://github.com/apache/iotdb/pull/7943


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@iotdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org