You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by da...@apache.org on 2020/08/07 11:31:21 UTC

[lucene-solr] branch jira/SOLR-14684 created (now 1b37c98)

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

datcm pushed a change to branch jira/SOLR-14684
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git.


      at 1b37c98  SOLR-14684: CloudExitableDirectoryReaderTest failing about 25% of the time

This branch includes the following new commits:

     new 1b37c98  SOLR-14684: CloudExitableDirectoryReaderTest failing about 25% of the time

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[lucene-solr] 01/01: SOLR-14684: CloudExitableDirectoryReaderTest failing about 25% of the time

Posted by da...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 1b37c981a0ced4876455c9e5effa488d71b70160
Author: Cao Manh Dat <da...@apache.org>
AuthorDate: Fri Aug 7 18:30:54 2020 +0700

    SOLR-14684: CloudExitableDirectoryReaderTest failing about 25% of the time
---
 .../src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java  | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java
index c1e6af7..f565c03 100644
--- a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java
+++ b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java
@@ -18,6 +18,7 @@
 package org.apache.solr.client.solrj.impl;
 
 import java.io.IOException;
+import java.lang.invoke.MethodHandles;
 import java.lang.ref.WeakReference;
 import java.net.ConnectException;
 import java.net.MalformedURLException;
@@ -54,12 +55,15 @@ import org.apache.solr.common.params.SolrParams;
 import org.apache.solr.common.util.ExecutorUtil;
 import org.apache.solr.common.util.NamedList;
 import org.apache.solr.common.util.SolrNamedThreadFactory;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.slf4j.MDC;
 
 import static org.apache.solr.common.params.CommonParams.ADMIN_PATHS;
 
 public abstract class LBSolrClient extends SolrClient {
 
+  private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
   // defaults
   protected static final Set<Integer> RETRY_CODES = new HashSet<>(Arrays.asList(404, 403, 503, 500));
   private static final int CHECK_INTERVAL = 60 * 1000; //1 minute between checks
@@ -155,6 +159,7 @@ public abstract class LBSolrClient extends SolrClient {
       this.req = req;
       this.zombieServers = zombieServers;
       this.timeAllowedNano = getTimeAllowedInNanos(req.getRequest());
+      log.info("TimeAllowedNano:{}", this.timeAllowedNano);
       this.timeOutTime = System.nanoTime() + timeAllowedNano;
       fetchNext();
     }
@@ -213,7 +218,8 @@ public abstract class LBSolrClient extends SolrClient {
       if (previousEx == null) {
         suffix = ":" + zombieServers.keySet();
       }
-      if (isTimeExceeded(timeAllowedNano, timeOutTime)) {
+      // Skipping check time exceeded for the first request
+      if (previousEx != null && isTimeExceeded(timeAllowedNano, timeOutTime)) {
         throw new SolrServerException("Time allowed to handle this request exceeded"+suffix, previousEx);
       }
       if (serverStr == null) {