Github user traflm commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1688#discussion_r223166271
--- Diff: core/sql/optimizer/ValueDesc.cpp ---
@@ -4555,7 +4555,7 @@ void ValueIdSet::unparse(NAString &result,
NAString connectorText;
- if ((form == MVINFO_FORMAT) || (form == QUERY_FORMAT))
+ if ((form == MVINFO_FORMAT) || (form == QUERY_FORMAT) || (form ==
CONNECT_BY_FORMAT))
--- End diff --
CONNECT_BY_FORMAT has only one difference from QUERY_FORMAT, that the
column name only has the last part, not qualified.
The reason is: the column name will be used in the work() method of
CONNECT_BY, where alias of the table name is lost. But we can make sure, there
is only one table in that query in the work() method, so only the column name
is required.
---
