[
https://issues.apache.org/jira/browse/HDDS-1441?focusedWorklogId=228578&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-228578
]
ASF GitHub Bot logged work on HDDS-1441:
----------------------------------------
Author: ASF GitHub Bot
Created on: 16/Apr/19 17:42
Start Date: 16/Apr/19 17:42
Worklog Time Spent: 10m
Work Description: swagle commented on issue #745: HDDS-1441. Remove usage
of getRetryFailureException. (swagle)
URL: https://github.com/apache/hadoop/pull/745#issuecomment-483774084
Thanks, @bshashikant for your review comments. Updated the patch with
suggested changes.
Although, wouldn't it be better to have a generic exception class in Ratis
that can have derived classes on the server side? It would allow separation of
concerns for server and client vs now any new type exception will break the
client side code in at runtime. Its not like we are doing anything with the
exception, only wrapping it up.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 228578)
Time Spent: 0.5h (was: 20m)
> Remove usage of getRetryFailureException
> ----------------------------------------
>
> Key: HDDS-1441
> URL: https://issues.apache.org/jira/browse/HDDS-1441
> Project: Hadoop Distributed Data Store
> Issue Type: Improvement
> Reporter: Arpit Agarwal
> Assignee: Siddharth Wagle
> Priority: Major
> Labels: pull-request-available
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> Per [~szetszwo]'s comment on RATIS-518, we can remove the usage of
> getRetryFailureException.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
