andygrove commented on PR #1066: URL: https://github.com/apache/datafusion-ballista/pull/1066#issuecomment-2407734270
> 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. Honestly, this is too big for me to do a comprehensive review within the limited weekend time that I have available. It seems that some very unrelated changes would be easy to split out, so we can start with those first to at least reduce the size of this PR? For example, these could be separate smaller PRs that would be easy to review: - Remove UI - Remove Python - Remove Helm - Clean up CI - Clean up dependencies We could get through these very quickly, I think. -- 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]
