Skip to content

Conversation

@zeroshade
Copy link
Member

fixes #2793 by checking for a concurrency of 0 and setting it to unlimited.

@github-actions github-actions bot added this to the ADBC Libraries 19 milestone May 9, 2025
Copy link
Member Author

@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.

@davlee1972 can you take a look at this to confirm it fixes your issue?

@davlee1972
Copy link

I don't really have a GoLang env on my company linux box to build the Go driver with the changes..

@zeroshade
Copy link
Member Author

@davlee1972 If I provide a linux build of the driver here, would you be able to use the adbc_driver_manager to load it and test?

@davlee1972
Copy link

yes.. I have both linux and windows environments available..

@zeroshade
Copy link
Member Author

Here you go, you can unzip this and it contains a linux_amd64 build of the snowflake driver. You should be able to initialize and load it via the driver manager.

libadbc_driver_snowflake.so.107.0.zip

@zeroshade
Copy link
Member Author

Any luck @davlee1972 ?

@davlee1972
Copy link

davlee1972 commented May 21, 2025

I'm running into some infrastructure issues locating a RHEL 9 Linux box .

The highest glibc I have is 2.28 on RHEL 8..

This shared lib is built with 2.32.

@zeroshade
Copy link
Member Author

I haven't had the time lately, but I'll try building with a lower glibc this week so you can test it out @davlee1972

@davlee1972
Copy link

Thanks..

FYI For the rows processed reporting mismatches Snowflake is working on a fix..

"Engineering will be working on prioritizing this fix in Workspace v2."

@lidavidm
Copy link
Member

@zeroshade should we update the docs and just merge this for release? (We always do multiple COPY INTO and that is contrary to the docs)

@zeroshade
Copy link
Member Author

@lidavidm I would be in favor of that. Should I update the docs in this PR or make a new one?

@lidavidm
Copy link
Member

In this PR works

@zeroshade
Copy link
Member Author

Okay I'll update the docs in the morning tomorrow so we can get this merged

@zeroshade
Copy link
Member Author

docs updated

@lidavidm lidavidm merged commit 4fab33e into apache:main Jul 1, 2025
42 checks passed
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.

adbc.snowflake.statement.ingest_copy_concurrency = "0" blocks copy into

3 participants