Epicism commented on PR #1125: URL: https://github.com/apache/datafusion-ballista/pull/1125#issuecomment-2489582982
PoV: Shouldn't the Python code be in its own repo? Ballista is being redesigned as a library, and Python is an application of the library... On Wed, Nov 20, 2024 at 4:05 PM Marko Milenković ***@***.***> wrote: > I have mixed feelings about keeping that test which involves ballista > python, should we remove it and risk breaking python from time to time? > wdyt @Dandandan <https://github.com/Dandandan> @andygrove > <https://github.com/andygrove> > > — > Reply to this email directly, view it on GitHub > <https://github.com/apache/datafusion-ballista/pull/1125#issuecomment-2489539232>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ACNEIPJOQLOCMX5DNZEFQGL2BT2P7AVCNFSM6AAAAABSFBQZ3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBZGUZTSMRTGI> > . > You are receiving this because you are subscribed to this thread.Message > ID: ***@***.***> > -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
