You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[C++][FlightRPC][ODBC] StatementTest/1.TestChangingAppRowDescriptorRebindsColumns fails on CI #54645
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.
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.
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!!
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!
Activity
alinaliBQ commentedon Oct 19, 2026
Hi @vikrantpuppala would you be able to look into the
StatementTest/1.TestChangingAppRowDescriptorRebindsColumnstest 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
alinaliBQ commentedon Oct 19, 2026
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!!alinaliBQ commentedon Oct 19, 2026
I was able to fix the issue locally. In short the query
did not work with the dremio oss docker instance, because alias
valueis not accepted.Changing the query to:
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!
lidavidm commentedon Oct 20, 2026
Ah sorry...I missed the label. Thanks for catching this.
vikrantpuppala commentedon Oct 20, 2026
Thanks for flagging @alinaliBQ, taking a look at the test
alinaliBQ commentedon Oct 20, 2026
All good and thanks both!