arham0254A commented on PR #56664: URL: https://github.com/apache/spark/pull/56664#issuecomment-4783454045
>Hi @uros-b, thanks for the detailed review! Apologies for the blank description—I actually had it filled out, but I accidentally left my text inside the template's HTML comment tags (<!-- -->), so it was hidden. I've updated the description now so it's visible. My original intent was to address SPARK-56539, which reported an exit code of 0 for unrecognized spark-submit arguments. However, based on your notes about how SparkException propagates and how exitFn is mocked in the test suite, I clearly see how my approach was flawed and introduced a regression. I really appreciate you taking the time to explain the control flow here! I am absolutely committed to getting this right. I am going to take a step back, try to reproduce the exact conditions of the original issue to find the true root cause, and work on a proper fix. Thanks for your patience and guidance! -- 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]
