ref(connection): Remove lock_timeout on replication connection

[iambriccardo]

Summary by CodeRabbit

• Bug Fixes
  • Disabled lock timeout in ETL replication operations, allowing long-running operations to complete without timeout interruptions on busy databases.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated ETL replication connection timeout configuration by disabling the lock_timeout (changed from 30,000 to 0 milliseconds) to allow long-running operations during replication slot creation. Added documentation explaining that CREATE_REPLICATION_SLOT must wait for in-progress transactions to complete, which can take extended time on busy databases.

Changes

Cohort / File(s)	Summary
Replication Configuration & Documentation etl-config/src/shared/connection.rs	Changed ETL_REPLICATION_OPTIONS lock_timeout from 30000 to 0; expanded documentation comments to explain the disable rationale; updated corresponding test assertions to reflect the new timeout value

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: disabling lock_timeout on the replication connection by setting it to 0 instead of 30000.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coveralls]

Pull Request Test Coverage Report for Build 21433595385

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 40 unchanged lines in 4 files lost coverage.
• Overall coverage decreased (-0.009%) to 81.228%

---

Files with Coverage Reduction	New Missed Lines	%
etl-postgres/src/tokio/test_utils.rs	4	81.97%
etl/src/workers/table_sync.rs	4	89.62%
etl/src/replication/apply.rs	12	83.51%
etl/src/replication/client.rs	20	85.43%

Totals
Change from base Build 21402452125:	-0.009%
Covered Lines:	17386
Relevant Lines:	21404

---

💛 - Coveralls