Skip to content

Conversation

@howareyouman
Copy link
Contributor

@howareyouman howareyouman commented Jun 27, 2025

returning back function FlightSQLDriverInit
Fixes 3055.

@howareyouman howareyouman requested a review from zeroshade as a code owner June 27, 2025 14:17
@github-actions github-actions bot added this to the ADBC Libraries 19 milestone Jun 27, 2025
@howareyouman howareyouman changed the title adding back compatibility for function FlightSQLDriverInit bug(go/adbc/pkg/flightsql): adding back compatibility for function FlightSQLDriverInit Jun 27, 2025
@howareyouman howareyouman changed the title bug(go/adbc/pkg/flightsql): adding back compatibility for function FlightSQLDriverInit fix(go/adbc): adding back compatibility for function FlightSQLDriverInit Jun 27, 2025
Copy link
Member

@zeroshade zeroshade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably got lost when these were re-generated at some point. Should we put this function in a non-generated file so that we don't accidentally lose it again?

@howareyouman
Copy link
Contributor Author

Should we put this function in a non-generated file so that we don't accidentally lose it again?

Now we should put these 2 functions then - FlightSQLDriverInit, FlightSqlDriverInit.

@howareyouman
Copy link
Contributor Author

@zeroshade I will do it in the next PR

@howareyouman
Copy link
Contributor Author

@zeroshade could you please merge the PR? I cannot do it myself.

@zeroshade zeroshade merged commit ba2f0e0 into apache:main Jun 27, 2025
85 of 93 checks passed
@howareyouman
Copy link
Contributor Author

@zeroshade The pr with migration of these 2 functions to go driver code - here

zeroshade pushed a commit that referenced this pull request Jul 7, 2025
…3079)

Moving FlightSQLDriverInit function to a separate go file.
Resolving the request from the recent PR -
[link](#3056 (review))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Function FlightSQLDriverInit is not found

3 participants