aokolnychyi commented on code in PR #40707:
URL: https://github.com/apache/spark/pull/40707#discussion_r1169543497
##########
core/src/main/resources/error/error-classes.json:
##########
@@ -793,7 +793,7 @@
],
"sqlState" : "22003"
},
- "INVALID_BUCKET_FILE" : {
+ "TRANSIENT_INVALID_BUCKET_FILE" : {
Review Comment:
Optional: Instead of changing the error class name (since that error class
is shown in the error message), I'd consider adding a field to indicate whether
the error is transient (i.e. should be retried), similar to `sqlCode` we have
today. We would need more feedback from folks who worked on the error framework.
--
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]
