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

Create python bindings #34

Open
matthewmturner opened this issue Feb 1, 2022 · 7 comments
Open

Create python bindings #34

matthewmturner opened this issue Feb 1, 2022 · 7 comments

Comments

@matthewmturner
Copy link
Collaborator

No description provided.

@jimexist
Copy link

jimexist commented Feb 1, 2022

i guess this can be combined in datafusion-python repo

@matthewmturner
Copy link
Collaborator Author

@jimexist that would certainly decrease time to market. In the short run I think its a great idea.

and just to confirm, are you referring to make it part of the same datafusion python package? or a separate package like datafusion-s3?

@seddonm1 any thoughts?

@seddonm1
Copy link
Collaborator

seddonm1 commented Feb 1, 2022

Sorry I have zero knowledge of the Python bindings.

The only benefit of separating it from the datafusion-python repo may be if users don't want to install certain components (for example a GCP user may not need S3)?

@jimexist
Copy link

jimexist commented Feb 2, 2022

something we can consider is to make sure this repo's python API is compatible with the one from datafusion-python, and then allow it to be installed via extras:

In this way, users can do:

pip install datafusion datafusion[s3] datafusion[gcp]
@matthewmturner
Copy link
Collaborator Author

@jimexist i think thats a good idea. I will look into it.

@chitralverma
Copy link

+1 for this!

@schapala-hm
Copy link

+1 for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants