You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@uniffle.apache.org by "advancedxy (via GitHub)" <gi...@apache.org> on 2023/03/22 02:04:49 UTC

[GitHub] [incubator-uniffle] advancedxy commented on issue #750: [Improvement] Fetch Failed related exceptions should have a dedicated RssException class

advancedxy commented on issue #750:
URL: https://github.com/apache/incubator-uniffle/issues/750#issuecomment-1478821294

   > > Currently, a lot of exception are all called RssException, which has no distinguish meanings.
   > 
   > Thanks for the improvement. There are also many raw use of `RuntimeException` need to be replaced.
   
   Yes, it's definitely worth refactor. Would you mind create an issue for this? 


-- 
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: dev-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org