Merge pull request #1 from arpan14/retryerror

fix: prevent illegal negative timeout values into thread sleep() method while retrying exceptions in unit tests.
arpan14 · Feb 8, 2023 · 49a85df · 49a85df
2 parents 510d0fb + edc5bbf
commit 49a85df
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/google-cloud-spanner/src/test/java/com/google/cloud/spanner/it/ITClosedSessionTest.java b/google-cloud-spanner/src/test/java/com/google/cloud/spanner/it/ITClosedSessionTest.java
@@ -251,7 +251,10 @@ public void testTransactionManager() throws InterruptedException {
             break;
           }
         } catch (AbortedException e) {
-          Thread.sleep(e.getRetryDelayInMillis());
+          long retryDelayInMillis = e.getRetryDelayInMillis();
+          if(retryDelayInMillis > 0) {
+            Thread.sleep(retryDelayInMillis);
+          }
           txn = manager.resetForRetry();
         }
       }