You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-issues@hadoop.apache.org by "Jing Zhao (JIRA)" <ji...@apache.org> on 2015/05/05 00:52:07 UTC
[jira] [Commented] (HDFS-8321) CacheDirectives and CachePool
operations should throw RetriableException in safemode
[ https://issues.apache.org/jira/browse/HDFS-8321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14527508#comment-14527508 ]
Jing Zhao commented on HDFS-8321:
---------------------------------
+1 pending Jenkins.
> CacheDirectives and CachePool operations should throw RetriableException in safemode
> ------------------------------------------------------------------------------------
>
> Key: HDFS-8321
> URL: https://issues.apache.org/jira/browse/HDFS-8321
> Project: Hadoop HDFS
> Issue Type: Bug
> Reporter: Haohui Mai
> Assignee: Haohui Mai
> Attachments: HDFS-8321.000.patch
>
>
> Operations such as {{addCacheDirectives()}} throws {{SafeModeException}} when the NN is in safemode:
> {code}
> if (isInSafeMode()) {
> throw new SafeModeException(
> "Cannot add cache directive", safeMode);
> }
> {code}
> While other NN operations throws {{RetriableException}} when HA is enabled:
> {code}
> void checkNameNodeSafeMode(String errorMsg)
> throws RetriableException, SafeModeException {
> if (isInSafeMode()) {
> SafeModeException se = new SafeModeException(errorMsg, safeMode);
> if (haEnabled && haContext != null
> && haContext.getState().getServiceState() == HAServiceState.ACTIVE
> && shouldRetrySafeMode(this.safeMode)) {
> throw new RetriableException(se);
> } else {
> throw se;
> }
> }
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)