fix: address CodeRabbit review feedback

- Remove 'no transaction in progress' from Vertica transient patterns
  (it's a transaction-state notice, not a transient connection error)
- Hardcode dbt-data-reliability-ref to 'vertica-compat' for the Vertica
  matrix entry until PR #963 merges into master

Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>

Author: devin-ai-integration[bot]

.github/workflows/test-all-warehouses.yml

@@ -108,7 +108,9 @@ jobs:
     with:
       warehouse-type: ${{ matrix.warehouse-type }}
       elementary-ref: ${{ inputs.elementary-ref || ((github.event_name == 'pull_request_target' || github.event_name == 'pull_request') && github.event.pull_request.head.sha) || '' }}
-      dbt-data-reliability-ref: ${{ inputs.dbt-data-reliability-ref }}
+      # Vertica support lives on the vertica-compat branch of dbt-data-reliability
+      # until PR #963 merges.  After that, remove this override.
+      dbt-data-reliability-ref: ${{ inputs.dbt-data-reliability-ref || (matrix.warehouse-type == 'vertica' && 'vertica-compat') || '' }}
       dbt-version: ${{ matrix.dbt-version }}
       generate-data: ${{ inputs.generate-data || false }}
     secrets: inherit

elementary/clients/dbt/transient_errors.py

@@ -124,7 +124,6 @@
         "connection timed out",
         "could not connect to the server",
         "ssl syscall error",
-        "no transaction in progress",
     ),
 }