Skip to content

[C++][FlightRPC][ODBC] StatementTest/1.TestChangingAppRowDescriptorRebindsColumns fails on CI #54645

Open

Description

@alinaliBQ

Describe the bug, including details regarding any error messages, version, and platform.

StatementTest/1.TestChangingAppRowDescriptorRebindsColumns in the ODBC test suite is failing in the remote test suite, separate from the local mock test suite.

The failure was introduced by #51342 (GH-51341). Link to failure: https://github.com/apache/arrow/actions/runs/39181261765/job/117834344797#step:7:15959

Component(s)

FlightRPC, C++

Activity

  1. alinaliBQ commented on Oct 19, 2026

    @alinaliBQ
    CollaboratorAuthor

    Hi @vikrantpuppala would you be able to look into the StatementTest/1.TestChangingAppRowDescriptorRebindsColumns test failure?

    The failure was not caught before merge because the C++ Extra label was not applied automatically in the PR, so the ODBC tests were not run, which is a separate issue that I can look into as part of #54638.

    cc @lidavidm @amoeba @raulcd

  2. alinaliBQ commented on Oct 19, 2026

    @alinaliBQ
    CollaboratorAuthor

    I found in apache/arrow#50133 that the extra labels for C++ have been moved to a manual labeling process. Please apply the the CI: Extra: C++ label to the ODBC-related PRs when opening them and spread the word to new contributors, thank you very much!!

  3. alinaliBQ commented on Oct 19, 2026

    @alinaliBQ
    CollaboratorAuthor

    I was able to fix the issue locally. In short the query

       SELECT 1 AS value
       UNION ALL SELECT 2
       UNION ALL SELECT 3;
    

    did not work with the dremio oss docker instance, because alias value is not accepted.

    Changing the query to:

       SELECT 1 AS val
       UNION ALL SELECT 2
       UNION ALL SELECT 3;
    

    fixed the issue. This should be a one-liner fix and I can put up a PR for this soon (as part of #54638), but feel free to take it on if anyone else is interested!

  4. lidavidm commented on Oct 20, 2026

    @lidavidm
    Member

    Ah sorry...I missed the label. Thanks for catching this.

  5. vikrantpuppala commented on Oct 20, 2026

    @vikrantpuppala
    Contributor

    Thanks for flagging @alinaliBQ, taking a look at the test

  6. alinaliBQ commented on Oct 20, 2026

    @alinaliBQ
    CollaboratorAuthor

    All good and thanks both!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions