madrob commented on a change in pull request #207:
URL: https://github.com/apache/solr/pull/207#discussion_r666531293



##########
File path: solr/core/src/java/org/apache/solr/core/SolrCore.java
##########
@@ -2339,8 +2339,7 @@ public IndexFingerprint 
getIndexFingerprint(SolrIndexSearcher searcher, LeafRead
 
       final SolrIndexSearcher currSearcher = currSearcherHolder == null ? null 
: currSearcherHolder.get();
 
-      @SuppressWarnings({"rawtypes"})
-      Future future = null;
+      Future<Void> future = null;

Review comment:
       Yea, that's part of why I have the comment elsewhere that this 
could/should be an AtomicReference, but I don't want to change it to that 
pattern in this PR.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to