Skip to content

Conversation

@lidavidm
Copy link
Member

@lidavidm lidavidm commented Jul 2, 2025

No description provided.

@lidavidm lidavidm marked this pull request as ready for review July 2, 2025 02:34
@lidavidm lidavidm requested a review from mbrobbel July 2, 2025 02:34
@github-actions github-actions bot added this to the ADBC Libraries 19 milestone Jul 2, 2025
Copy link
Member

@amoeba amoeba left a comment

Choose a reason for hiding this comment

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

For consistency, should we add a Rust section in https://arrow.apache.org/adbc/current/driver/installation.html with basically this?

Rust
====

Add the appropriate driver package as a dependency:

..  code-block:: shell
   cargo add adbc_datafusion

lidavidm and others added 2 commits July 2, 2025 11:50
Co-authored-by: Bryce Mecum <petridish@gmail.com>
@lidavidm
Copy link
Member Author

lidavidm commented Jul 2, 2025

Good point, I added a quick section in the installation page

Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

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

Thanks!

I thought it might help to mention that the following use are needed.

use adbc_core::{Connection, Database, Driver, Statement};

@lidavidm
Copy link
Member Author

lidavidm commented Jul 2, 2025

Fixed!

Co-authored-by: Daijiro Fukuda <fukuda@clear-code.com>
@lidavidm lidavidm merged commit c9fc5b9 into apache:main Jul 2, 2025
26 checks passed
@lidavidm lidavidm deleted the docs branch July 2, 2025 04:15
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.

3 participants