Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document Python async connection and node sync connections #414

Closed
2 tasks done
prrao87 opened this issue Mar 17, 2025 · 2 comments
Closed
2 tasks done

Document Python async connection and node sync connections #414

prrao87 opened this issue Mar 17, 2025 · 2 comments
Assignees
Labels
next-release Include for next release

Comments

@prrao87
Copy link
Member

prrao87 commented Mar 17, 2025

People should be aware that we will now provide a Python async and node.js sync (to go with their pre-existing sync/async counterparts) clients, so they can decide what works for their application.

@prrao87 prrao87 self-assigned this Mar 17, 2025
@ray6080 ray6080 mentioned this issue Mar 20, 2025
45 tasks
@ray6080 ray6080 added the next-release Include for next release label Mar 20, 2025
@royi-luo
Copy link
Contributor

Just a heads-up reading from dataframes currently don't work with the Python async API.

For example the following code will give an error Binder exception: Variable df is not in scope.

tab = pa.Table.from_arrays(
        [
            [1, 2, 3],
            ["a", "b", "c"],
            [True, False, None],
        ],
        names=["col1", "col2", "col3"],
    )
result = await conn.execute("LOAD FROM tab RETURN *")

My guess is that this is because async connections execute queries using a separate thread from the calling thread, and thus the dataframe isn't scope for the executing thread. Not sure if there's an easy fix for this but we should document this to start.

@ray6080
Copy link
Contributor

ray6080 commented Mar 27, 2025

@royi-luo nice catch. I moved this to an issue on main repo kuzudb/kuzu#5139

@prrao87 prrao87 closed this as completed Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next-release Include for next release
Projects
None yet
Development

No branches or pull requests

4 participants