Implement Python bindings for BallistaContext about datafusion-ballista HOT 12 CLOSED

@andygrove @alamb @jorgecarleitao thanks for comment.
Will start implementation in following branch apache/datafusion#988
(will request for review when ready 🙇 )

from datafusion-ballista.

Thank you @djKooks that would be great

from datafusion-ballista.

@andygrove
Would it be okay to put like following?

...
ballista/
   - rust/
   - ui/
   - python/     <- create binding here
datafusion/
datafusion-cli/
...

or should I update current datafusion python binding inside existing python/ directory?

from datafusion-ballista.

I agree that this makes the most sense 👍

Out of curiosity, do the bindings come with the client and executors, or just the client?

from datafusion-ballista.

I agree having a separate python binding for Ballista in the location suggested by @djKooks in #15 makes sense to me

from datafusion-ballista.

@alamb @jorgecarleitao @andygrove thanks for suggestion 🙇

do the bindings come with the client and executors, or just the client?

I think it will be enough to do with client only in first step, but do you have any more suggestion?

from datafusion-ballista.

@andygrove hope to work on this, if you don't have any other plan.

from datafusion-ballista.

Yes, I think that makes sense.

from datafusion-ballista.

I would be interested to hear what others think though. @alamb @Dandandan @jorgecarleitao @houqp do you have an opinion on this?

from datafusion-ballista.

I do not have any more to add here -- since I don't use the python bindings myself I don't have a lot to offer with specifics

from datafusion-ballista.

Related to #58

from datafusion-ballista.

This has now been implemented.

from datafusion-ballista.