wangzhaohui created HDFS-15780:
----------------------------------

             Summary: RBF: Delete  unnecessary var in stateStoreService
                 Key: HDFS-15780
                 URL: https://issues.apache.org/jira/browse/HDFS-15780
             Project: Hadoop HDFS
          Issue Type: Bug
            Reporter: wangzhaohui


{code:java}
public void refreshCaches(boolean force) {
  boolean success = true;
  if (isDriverReady()) {
    List<StateStoreCache> cachesToUpdate = new LinkedList<>();
    cachesToUpdate.addAll(cachesToUpdateInternal);
    cachesToUpdate.addAll(cachesToUpdateExternal);
    for (StateStoreCache cachedStore : cachesToUpdate) {
      String cacheName = cachedStore.getClass().getSimpleName();
      boolean result = false;
      try {
        result = cachedStore.loadCache(force);
      } catch (IOException e) {
        LOG.error("Error updating cache for {}", cacheName, e);
        result = false;
      }
      if (!result) {
        success = false;
        LOG.error("Cache update failed for cache {}", cacheName);
      }
    }
  } else {
    success = false;
    LOG.info("Skipping State Store cache update, driver is not ready.");
  }
  if (success) {
    // Uses local time, not driver time.
    this.cacheLastUpdateTime = Time.now();
  }
}
{code}
The "success" may be unnecessary,we can use "result" instead of it, this can 
reduce the use of unnecessary variables.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to