You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have searched in the issues and found no similar issues.
Describe the bug
We need to uniformly throw RssException for external callers, making it convenient for them to perform unified exception handling.
For example , as described in #888, we can uniformly handle RssException in Spark's TaskSetManager.
Some custom exception classes in the rss-common module directly inherit from RuntimeException, instead of RssException. Since the rss-common module can be directly used by clients, we need to fix this issue.
Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?
Yes I am willing to submit a PR!
The text was updated successfully, but these errors were encountered:
rickyma
added a commit
to rickyma/incubator-uniffle
that referenced
this issue
Feb 28, 2024
#1550)
### What changes were proposed in this pull request?
Make custom exception classes in the `rss-common` module inherit from `RssException`.
### Why are the changes needed?
For #1549
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing UTs.
Code of Conduct
Search before asking
Describe the bug
We need to uniformly throw
RssException
for external callers, making it convenient for them to perform unified exception handling.For example , as described in #888, we can uniformly handle
RssException
in Spark'sTaskSetManager
.Some custom exception classes in the
rss-common
module directly inherit fromRuntimeException
, instead ofRssException
. Since therss-common
module can be directly used by clients, we need to fix this issue.Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?
The text was updated successfully, but these errors were encountered: