apacheGH-30622: [GAC-47] Move Timestream ODBC driver seed under cpp/src/flightsql_odbc #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Updated workflows to accommodate this change.
Ubuntu 20.04 Debug Build Test Results Check
|
|
Is the folder structure correct arrow\cpp\src\flightsql_odbc\amazon-timestream-odbc-driver? |
@affonsov Yes, I put it like that so later we could add |
Windows 2022 (x64) Debug Build Code Coverage Test Results Check
|
MacOS Big Sur 11 Debug Build Code Coverage Unit and Integration Test Result
|
affonsov
approved these changes
Dec 7, 2023
alinaliBQ
pushed a commit
that referenced
this pull request
Dec 7, 2023
Signed-off-by: forestmvey <[email protected]>
alinaliBQ
pushed a commit
that referenced
this pull request
Dec 13, 2023
Signed-off-by: forestmvey <[email protected]>
alinaliBQ
pushed a commit
that referenced
this pull request
Dec 18, 2023
Signed-off-by: forestmvey <[email protected]>
alinaliBQ
pushed a commit
that referenced
this pull request
Dec 18, 2023
Signed-off-by: forestmvey <[email protected]>
alinaliBQ
pushed a commit
that referenced
this pull request
Dec 20, 2023
…mation improvement (#4) * fix occasional test failure * document update * add check for credentials leak Fix wrong linux package name and remove unneeded files (#2) * code uploading * fix cppcheck failure (#1) * fix cppcheck failure * fix build script not running and credentials could not be fetched * restore configure-aws-credentials to v2 * add execute permission for shell scripts * remove bash for mac build scripts * trigger all tests * verify signer works fine * fix singer could not find the installer package * change signed installer name * fix linux package wrong name issue * restore signer flag * add .gitignore * remove unneeded files * remove unneeded files * address comments * add version to installer name * remove not needed file fix cppcheck failure (#1) * fix cppcheck failure * fix build script not running and credentials could not be fetched * restore configure-aws-credentials to v2 * add execute permission for shell scripts * remove bash for mac build scripts * trigger all tests code uploading Initial commit fix bracket missing (#5) Pass driver info to Timestream using user agent (#6) * pass driver info to Timestream using user agent * add log for user agent setting Fixing awslabs links. (#8) Signed-off-by: forestmvey <[email protected]> Move AWS TS ODBC Driver under flight sql subdirectory Enable Windows builds, workflow files are based on Amazon Timestream ODBC driver seed disable integration tests from running. The win-build.yml file is copied from Amazon Timestream ODBC driver code. Fix CMAKE code formats Note that rest of lint errors are not solved yet. Fix python code doc errors Enable linux build, the file is copied from Amazon Timestream ODBC seed * integration tests are disabled. * allow build scripts to be run through `chmod`. Enable macOS builds, workflow files are copied from Amazon TS ODBC seed Move Timestream ODBC driver seed under cpp/src/flightsql_odbc * Updated workflows to accommodate this change. Remove Timestream-related code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rationale for this change
What changes are included in this PR?
Move Timestream ODBC driver seed under cpp/src/flightsql_odbc
Are these changes tested?
N/A, since code is just moved. Updated workflows to accommodate this change.
Are there any user-facing changes?
No.