[ 
https://issues.apache.org/jira/browse/HDFS-13330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16414169#comment-16414169
 ] 

Gabor Bota edited comment on HDFS-13330 at 3/27/18 4:58 AM:
------------------------------------------------------------

I've started watching HDFS-8979 in case it won't get reverted and I need to fix 
the retry code.


was (Author: gabor.bota):
I'm started watching HDFS-8979 in case it won't get reverted and I need to fix 
the retry code.

> ShortCircuitCache#fetchOrCreate never retries
> ---------------------------------------------
>
>                 Key: HDFS-13330
>                 URL: https://issues.apache.org/jira/browse/HDFS-13330
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Wei-Chiu Chuang
>            Assignee: Gabor Bota
>            Priority: Minor
>         Attachments: HDFS-13330.001.patch
>
>
> The follow do .. while(false) loop seems useless to me. The code intended to 
> retry but it never worked. Let's fix it.
> {code:java:title=ShortCircuitCache#fetchOrCreate}
> ShortCircuitReplicaInfo info = null;
> do {
>   if (closed) {
>     LOG.trace("{}: can't fethchOrCreate {} because the cache is closed.",
>         this, key);
>     return null;
>   }
>   Waitable<ShortCircuitReplicaInfo> waitable = replicaInfoMap.get(key);
>   if (waitable != null) {
>     try {
>       info = fetch(key, waitable);
>     } catch (RetriableException e) {
>       LOG.debug("{}: retrying {}", this, e.getMessage());
>     }
>   }
> } while (false);{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to