fix: only try batch requests with non-empty entries

sqs_workers/queue.py

@@ -148,24 +148,26 @@ def _handle_processed(self, messages_with_success: Iterable[Tuple[Any, bool]]):
                         }
                     )
 
-            ack_response = queue.delete_messages(Entries=entries_to_ack)
+            if len(entries_to_ack) > 0:
+                ack_response = queue.delete_messages(Entries=entries_to_ack)
 
-            if ack_response.get("Failed"):
-                logger.warning(
-                    "Failed to delete processed messages from queue",
-                    extra={"queue": self.name, "failures": ack_response["Failed"]},
-                )
-
-            viz_response = queue.change_message_visibility_batch(
-                Entries=entries_to_change_viz,
-            )
+                if ack_response.get("Failed"):
+                    logger.warning(
+                        "Failed to delete processed messages from queue",
+                        extra={"queue": self.name, "failures": ack_response["Failed"]},
+                    )
 
-            if viz_response.get("Failed"):
-                logger.warning(
-                    "Failed to change visibility of messages which failed to process",
-                    extra={"queue": self.name, "failures": viz_response["Failed"]},
+            if len(entries_to_change_viz) > 0:
+                viz_response = queue.change_message_visibility_batch(
+                    Entries=entries_to_change_viz,
                 )
 
+                if viz_response.get("Failed"):
+                    logger.warning(
+                        "Failed to change visibility of messages which failed to process",
+                        extra={"queue": self.name, "failures": viz_response["Failed"]},
+                    )
+
         return result
 
     def process_message(self, message: Any) -> bool: