[GitHub] [spark] MaxGekk commented on pull request #36500: [SPARK-39164][SQL] Wrap asserts/illegal state exceptions by the INTERNAL_ERROR exception in actions

MaxGekk commented on PR #36500: URL: https://github.com/apache/spark/pull/36500#issuecomment-1125412163 Merging to master. Thank you, @cloud-fan for review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [spark] MaxGekk commented on pull request #36500: [SPARK-39164][SQL] Wrap asserts/illegal state exceptions by the INTERNAL_ERROR exception in actions

MaxGekk commented on PR #36500: URL: https://github.com/apache/spark/pull/36500#issuecomment-1125377146 All GAs except of [Run Docker integration tests](https://github.com/MaxGekk/spark/runs/6411188536?check_suite_focus=true) passed. I will re-run and wait for the docker tests only. --

[GitHub] [spark] MaxGekk commented on pull request #36500: [SPARK-39164][SQL] Wrap asserts/illegal state exceptions by the INTERNAL_ERROR exception in actions

MaxGekk commented on PR #36500: URL: https://github.com/apache/spark/pull/36500#issuecomment-1125097534 > shall we handle internal errors in the query compilation phase? like parser, analyzer, optimizer, planner. Yep, I plan to wrap asserts/illegal state exceptions by internal errors

[GitHub] [spark] MaxGekk commented on pull request #36500: [SPARK-39164][SQL] Wrap asserts/illegal state exceptions by the INTERNAL_ERROR exception in actions

MaxGekk commented on PR #36500: URL: https://github.com/apache/spark/pull/36500#issuecomment-1124832137 @srielau @panbingkun @lvshaokang Please, take a look at the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use