bruno-roustant commented on a change in pull request #108:
URL: https://github.com/apache/solr/pull/108#discussion_r651622909
##########
File path:
solr/contrib/blob-directory/src/java/org/apache/solr/blob/BlobDirectory.java
##########
@@ -61,20 +62,33 @@
private final Object lock = new Object();
private volatile boolean isOpen;
- public BlobDirectory(Directory delegate, String blobDirPath,
BlobStoreConnection blobStoreConnection) throws IOException {
+ public BlobDirectory(Directory delegate, String blobDirPath, BlobRepository
blobRepository) throws IOException {
super(delegate);
this.blobDirPath = blobDirPath;
- this.blobStoreConnection = blobStoreConnection;
+ this.blobRepository = blobRepository;
pullMissingFilesFromRepo();
}
private void pullMissingFilesFromRepo() throws IOException {
Set<String> localFileNames = new HashSet<>(Arrays.asList(in.listAll()));
Review comment:
I'll add both a comment and javadoc for the fileFilter param of the pull
method.
Currently the file selection is done on the current thread. Then the files
are pulled by multiple threads.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
