@Nullable
 private T getFromCache(@NotNull Integer commitId) {
   T details = myCache.get(commitId);
   if (details != null) {
     if (details instanceof LoadingDetails) {
       if (((LoadingDetails) details).getLoadingTaskIndex()
           <= myCurrentTaskIndex - MAX_LOADING_TASKS) {
         // don't let old "loading" requests stay in the cache forever
         myCache.remove(commitId);
         return null;
       }
     }
     return details;
   }
   return getFromAdditionalCache(commitId);
 }
  @Override
  @NotNull
  public T getCommitData(@NotNull Integer hash, @NotNull Iterable<Integer> neighbourHashes) {
    assert EventQueue.isDispatchThread();
    T details = getFromCache(hash);
    if (details != null) {
      return details;
    }

    runLoadCommitsData(neighbourHashes);

    T result = myCache.get(hash);
    assert result
        != null; // now it is in the cache as "Loading Details" (runLoadCommitsData puts it there)
    return result;
  }