fix: Drop tenacity retry around connect

Wrapping retry for essentially *all* exceptions can lead to repeated
failures which will not magically be handled. retry should be instead
used more cautiously only for known failures.

insights/insights/doctype/insights_data_source/sources/base_database.py

@@ -5,7 +5,6 @@
 
 import frappe
 from sqlalchemy.sql import text
-from tenacity import retry, retry_if_exception_type, stop_after_attempt, wait_fixed
 
 from insights.insights.doctype.insights_table_import.insights_table_import import (
     InsightsTableImport,
@@ -88,12 +87,6 @@ def test_connection(self):
             res = connection.execute(text("SELECT 1"))
             return res.fetchone()
 
-    @retry(
-        retry=retry_if_exception_type((DatabaseParallelConnectionError,)),
-        stop=stop_after_attempt(5),
-        wait=wait_fixed(1),
-        reraise=True,
-    )
     def connect(self):
         try:
             return self.engine.connect()