keith-turner commented on code in PR #6025:
URL: https://github.com/apache/accumulo/pull/6025#discussion_r2737732898


##########
server/base/src/main/java/org/apache/accumulo/server/fs/FileManager.java:
##########
@@ -386,6 +388,7 @@ private void releaseReaders(KeyExtent tablet, 
List<FileSKVIterator> readers,
 
   }
 
+  @NotThreadSafe
   static class FileDataSource implements DataSource {

Review Comment:
   This is not ThreadSafe, however it may be used by multiple threads but all 
usages of it seem to be inside synchronized FileManager method so it would not 
need its own sync.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to