Skip to content

[C++][FlightRPC][ODBC] Add SQLGetInfo missing tests #50828

Closed

Description

@alinaliBQ

Describe the enhancement requested

There is a gap that 2 tests are missing from the original implementation:

  • TEST_F(ConnectionInfoRemoteTest, TestSQLGetInfoCreateTable) for SQL_CREATE_TABLE
  • TEST_F(ConnectionInfoRemoteTest, TestSQLGetInfoCreateSchema) for SQL_CREATE_SCHEMA
    If the expected value from the remote is same as the mock server, then we can merge the remote test with the mock test by using TYPED_TEST(ConnectionInfoHandleTest.

Theoretically these tests should also return 0 to signal table and schema DDL is not supported in Dremio instance. It also explains why we were only seeing inconsistencies in SQL_DROP_TABLE and SQL_DROP_SCHEMA in #49482.

Component(s)

C++, FlightRPC

Activity

  1. vanshaj2023 commented on Aug 6, 2026

    @vanshaj2023
    Contributor

    take

  2. added a commit that references this issue on Aug 10, 2026
  3. added a commit that references this issue on Oct 14, 2026
  4. added a commit that references this issue on Oct 15, 2026
  5. added this to the 27.0.0 milestone on Oct 15, 2026
  6. lidavidm commented on Oct 15, 2026

    @lidavidm
    Member

    Issue resolved by pull request 50838
    #50838

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions