[
https://issues.apache.org/jira/browse/CALCITE-6343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17831113#comment-17831113
]
Julian Hyde commented on CALCITE-6343:
--------------------------------------
Your PR looks fine, but I would like you to add a simple SQL test -- maybe in
{{measure.iq}} or SqlValidatorTest -- that demonstrates the derived type. If
you ask me to read the test you wrote tomorrow, I wouldn't be able to tell you
what it's testing.
> AS alias operator strips MEASUREness from measures
> --------------------------------------------------
>
> Key: CALCITE-6343
> URL: https://issues.apache.org/jira/browse/CALCITE-6343
> Project: Calcite
> Issue Type: Bug
> Components: core
> Affects Versions: 1.36.0
> Reporter: Barry Kelly
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.37.0
>
>
> CALCITE-5869 introduced a change which removes MEASURE when inferring the
> return type when an operator is applied.
> The {{AS}} keyword for aliases is implemented as an operator in the SQL AST.
> Using {{AS}} removes MEASURE when typing expressions that have an alias in
> the {{SELECT}} clause.
> Thus, the type of {{SELECT m}} and {{SELECT m AS m}} are different when {{m}}
> is a measure. This is not desirable.
> Proposed fix: don't change type when using the {{AS}} operator.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
