Add retry on 500 (#124)

firebolt-db · Nov 3, 2022 · 6be54fb · 6be54fb
1 parent 8e69802
commit 6be54fb
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -2,7 +2,7 @@ name: Release new version
 
 on:
   release:
-    types: [published]
+    types: [created]
 
 jobs:
   build:

diff --git a/src/main/java/com/firebolt/jdbc/client/config/RetryInterceptor.java b/src/main/java/com/firebolt/jdbc/client/config/RetryInterceptor.java
@@ -19,6 +19,7 @@
 public class RetryInterceptor implements Interceptor {
 
     private static final Set<Integer> RETRYABLE_RESPONSE_CODES = new HashSet<>(Arrays.asList(HTTP_CLIENT_TIMEOUT,
+            HTTP_INTERNAL_ERROR,
             HTTP_BAD_GATEWAY,
             HTTP_UNAVAILABLE,
             HTTP_GATEWAY_TIMEOUT));

diff --git a/src/test/java/com/firebolt/jdbc/client/config/RetryInterceptorTest.java b/src/test/java/com/firebolt/jdbc/client/config/RetryInterceptorTest.java
@@ -17,7 +17,7 @@
 class RetryInterceptorTest {
 
     @ParameterizedTest
-    @ValueSource(ints = {HTTP_CLIENT_TIMEOUT, HTTP_BAD_GATEWAY, HTTP_UNAVAILABLE,
+    @ValueSource(ints = {HTTP_CLIENT_TIMEOUT, HTTP_INTERNAL_ERROR, HTTP_BAD_GATEWAY, HTTP_UNAVAILABLE,
             HTTP_GATEWAY_TIMEOUT})
     void shouldRetryOnRetryableResponseCode(int arg) throws IOException {
         int retries = 3;
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,7 +2,7 @@ name: Release new version @@
     on:
       release:
-        types: [published]
+        types: [created]
     jobs:
       build:
@@ Expand Down @@