[
https://issues.apache.org/jira/browse/HDFS-16710?focusedWorklogId=797070&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-797070
]
ASF GitHub Bot logged work on HDFS-16710:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 02/Aug/22 00:41
Start Date: 02/Aug/22 00:41
Worklog Time Spent: 10m
Work Description: slfan1989 commented on PR #4670:
URL: https://github.com/apache/hadoop/pull/4670#issuecomment-1201880094
@ZanderXu We need to pay attention to checkstyle, and then see if the junit
test problem is related to this change. Thank you.
Issue Time Tracking
-------------------
Worklog Id: (was: 797070)
Time Spent: 50m (was: 40m)
> Remove redundant throw exceptions in org.apahce.hadoop.hdfs.server.namenode
> package
> -----------------------------------------------------------------------------------
>
> Key: HDFS-16710
> URL: https://issues.apache.org/jira/browse/HDFS-16710
> Project: Hadoop HDFS
> Issue Type: Improvement
> Reporter: ZanderXu
> Assignee: ZanderXu
> Priority: Major
> Labels: pull-request-available
> Time Spent: 50m
> Remaining Estimate: 0h
>
> When I read some class about HDFS NameNode, I found there are many redundant
> throw exception in org.apahce.hadoop.hdfs.server.namenode package, such as:
> {code:java}
> public synchronized void transitionToObserver(StateChangeRequestInfo req)
> throws ServiceFailedException, AccessControlException, IOException {
> checkNNStartup();
> nn.checkHaStateChange(req);
> nn.transitionToObserver();
> } {code}
> Because ServiceFailedException and AccessControlException is subClass of
> IOException, so I feel that ServiceFailedException and AccessControlException
> are redundant, so we can remove it to make code clearer, such as:
> {code:java}
> public synchronized void transitionToObserver(StateChangeRequestInfo req)
> throws IOException {
> checkNNStartup();
> nn.checkHaStateChange(req);
> nn.transitionToObserver();
> } {code}
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
