You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "kgyrtkirk (via GitHub)" <gi...@apache.org> on 2024/11/06 18:09:29 UTC
[PR] Align ScanQuery column order with its desired signature (druid)
kgyrtkirk opened a new pull request, #17457:
URL: https://github.com/apache/druid/pull/17457
`ScanQuery` [reorders its output columns](https://github.com/apache/druid/blob/9c25226e06da1eb7d3a47742dd4e65337585142c/sql/src/main/java/org/apache/druid/sql/calcite/rel/DruidQuery.java#L1677) to be in alphabetical ordering; this is not necessary - and depending on the names of the columns it may be different.
This is needed for `UnionQuery` to avoid incorrect results
Main change is a single line alteration in `DruidQuery` to use a `LinkedHashSet` instead of `TreeMap`.
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
Re: [PR] Align ScanQuery column order with its desired signature (druid)
Posted by "cryptoe (via GitHub)" <gi...@apache.org>.
cryptoe closed pull request #17457: Align ScanQuery column order with its desired signature
URL: https://github.com/apache/druid/pull/17457
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
Re: [PR] Align ScanQuery column order with its desired signature (druid)
Posted by "cryptoe (via GitHub)" <gi...@apache.org>.
cryptoe commented on PR #17457:
URL: https://github.com/apache/druid/pull/17457#issuecomment-2469947365
Closed in favour of https://github.com/apache/druid/pull/17463
--
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: commits-unsubscribe@druid.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org