You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by no...@apache.org on 2020/01/07 06:19:37 UTC

[lucene-solr] branch jira/solr14158 updated: precommit errors

This is an automated email from the ASF dual-hosted git repository.

noble pushed a commit to branch jira/solr14158
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/jira/solr14158 by this push:
     new a94275c  precommit errors
a94275c is described below

commit a94275c845a7562271a43081662ef5d4af19f09c
Author: noble <no...@apache.org>
AuthorDate: Tue Jan 7 17:19:20 2020 +1100

    precommit errors
---
 .../src/java/org/apache/solr/filestore/DistribPackageStore.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/solr/core/src/java/org/apache/solr/filestore/DistribPackageStore.java b/solr/core/src/java/org/apache/solr/filestore/DistribPackageStore.java
index ef142e0..3389bf4 100644
--- a/solr/core/src/java/org/apache/solr/filestore/DistribPackageStore.java
+++ b/solr/core/src/java/org/apache/solr/filestore/DistribPackageStore.java
@@ -104,7 +104,7 @@ public class DistribPackageStore implements PackageStore {
       this.path = path;
     }
 
-    public ByteBuffer getFileData(boolean validate) throws IOException {
+    ByteBuffer getFileData(boolean validate) throws IOException {
       if (fileData == null) {
         try (FileInputStream fis = new FileInputStream(getRealpath(path).toFile())) {
           fileData = SimplePostTool.inputStreamToByteArray(fis);
@@ -476,7 +476,7 @@ public class DistribPackageStore implements PackageStore {
       try {
         l = coreContainer.getZkController().getZkClient().getChildren(ZK_PACKAGESTORE+ path, null, true);
       } catch (KeeperException.NoNodeException e) {
-        // doe not matter
+        // does not matter
       }
       if (l != null && !l.isEmpty()) {
         List myFiles = list(path, s -> true);
@@ -488,7 +488,7 @@ public class DistribPackageStore implements PackageStore {
         }
       }
     } catch (Exception e) {
-      e.printStackTrace();
+      log.error("Could not refresh files in " +path, e);
     }
   }