Xiao-zhen-Liu commented on PR #5280: URL: https://github.com/apache/texera/pull/5280#issuecomment-4640782241
@kunwp1 No need to split. It's 23 files but one self-contained fix (scope large-binary storage and cleanup by execution id). The count is high because the JVM and Python sides each do the same thing with their own tests, plus some import changes from the Python refactor — not 23 separate changes. The parts also can't be merged on their own: naming the URI on the controller, reading it on the worker, and scoping create/delete only work together. Splitting them would leave `main` half-working, and the fix would no longer be revertible as one unit. @chenlica I'd keep it as one PR for that reason. Genuinely separable work — like the >1000-object delete I noted below — can go in its own follow-up. -- 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]
