[
https://issues.apache.org/jira/browse/HDFS-16710?focusedWorklogId=797222&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-797222
]
ASF GitHub Bot logged work on HDFS-16710:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 02/Aug/22 13:20
Start Date: 02/Aug/22 13:20
Worklog Time Spent: 10m
Work Description: ZanderXu commented on PR #4670:
URL: https://github.com/apache/hadoop/pull/4670#issuecomment-1202543041
@slfan1989 Sir, I have updated the patch to fix the checkstyle, please help
me review it again. Thanks
Issue Time Tracking
-------------------
Worklog Id: (was: 797222)
Time Spent: 1h 10m (was: 1h)
> 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: 1h 10m
> 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]
