fix(queue.job): Handle zero max_retries in job retry logic

Author: zamberjo

queue_job/jobrunner/runner.py

@@ -332,7 +332,7 @@ def _query_requeue_dead_jobs(self):
                 state=(
                     CASE
                         WHEN
-                            max_retries IS NOT NULL AND
+                            NULLIF(max_retries, 0) IS NOT NULL AND
                             retry IS NOT NULL AND
                             retry>max_retries
                         THEN 'failed'
@@ -346,7 +346,7 @@ def _query_requeue_dead_jobs(self):
                 exc_name=(
                     CASE
                         WHEN
-                            max_retries IS NOT NULL AND
+                            NULLIF(max_retries, 0) IS NOT NULL AND
                             retry IS NOT NULL AND
                             retry>max_retries
                         THEN 'JobFoundDead'
@@ -355,7 +355,7 @@ def _query_requeue_dead_jobs(self):
                 exc_info=(
                     CASE
                         WHEN
-                            max_retries IS NOT NULL AND
+                            NULLIF(max_retries, 0) IS NOT NULL AND
                             retry IS NOT NULL AND
                             retry>max_retries
                         THEN 'Job found dead after too many retries'