You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/06/08 01:22:00 UTC

[jira] [Updated] (CALCITE-5768) SqlImplementor.hasSortByOrdinal is broken

     [ https://issues.apache.org/jira/browse/CALCITE-5768?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

ASF GitHub Bot updated CALCITE-5768:
------------------------------------
    Labels: pull-request-available  (was: )

> SqlImplementor.hasSortByOrdinal is broken
> -----------------------------------------
>
>                 Key: CALCITE-5768
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5768
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: Will Noble
>            Assignee: Will Noble
>            Priority: Major
>              Labels: pull-request-available
>
> {{hasSortByOrdinal}} has an [improper early return|https://github.com/apache/calcite/blob/c4042a34ef054b89cec1c47fefcbc8689bad55be/core/src/main/java/org/apache/calcite/rel/rel2sql/SqlImplementor.java#L1902] that's causing incorrect results. It should only return early if the result is {{true}}. Otherwise, it should continue searching for ordinals.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)