milenkovicm commented on PR #1066: URL: https://github.com/apache/datafusion-ballista/pull/1066#issuecomment-2407480906
Will have something ready later today or tomorrow @andygrove, I was thinking to put proposal of the next steps on top of this PR, started already but did not have time to finish it yet. Now, this PR has many things removed, but not too many code changed, would you like to take it step by step and re-do this work again or we can just merge this as once off, without squashing commits so we can revert removal if needed @andygrove ? I understand that it is a bit of big PR. > This PR removes: > > - python support (datafusion python is the main effort, we should focus making it run on ballista) > - helm support (this can be contrib project) > - good part of ci directory (ci works without it) > - plugin sub-system (never finalized) > - caching support (this can be a contrib project) > - support for hdfs (library too old, there are newer library to be used) > - UI (can be a contrib project) > - made keda, rest-api, flight-sql optional (we could move them to contrib project) > - key value store is gone, only memory store remains > - removes some dependencies which were detected as not used -- 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]
