fix(engine_adapter): preserve pre_ping in with_settings

EngineAdapter.with_settings() did not forward the current _pre_ping
value to the cloned adapter, and the constructor defaults pre_ping to
False. Context.snapshot_evaluator clones the adapter this way, so
pre_ping: true silently did not apply to model evaluation or the
before_all/after_all environment statements, and a stale connection
after a long run failed instead of being re-established.

pre_ping is now carried over like the other settings, and remains
overridable via an explicit keyword argument.

Adds a regression test asserting the cloned adapter keeps pre_ping.

Signed-off-by: Sanjay Santhanam <51058514+Sanjays2402@users.noreply.github.com>

Author: Sanjays2402

sqlmesh/core/engine_adapter/base.py

@@ -178,6 +178,7 @@ def with_settings(self, **kwargs: t.Any) -> EngineAdapter:
             "query_execution_tracker": kwargs.pop(
                 "query_execution_tracker", self._query_execution_tracker
             ),
+            "pre_ping": kwargs.pop("pre_ping", self._pre_ping),
             **self._extra_config,
             **kwargs,
         }

tests/core/engine_adapter/test_base.py

@@ -3522,6 +3522,15 @@ def test_pre_ping(mocker: MockerFixture, make_mocked_engine_adapter: t.Callable)
     adapter._connection_pool.get().close.assert_called_once()
 
 
+def test_with_settings_preserves_pre_ping(make_mocked_engine_adapter: t.Callable):
+    adapter = make_mocked_engine_adapter(EngineAdapter, pre_ping=True)
+    assert adapter.with_settings()._pre_ping is True
+    assert adapter.with_settings(pre_ping=False)._pre_ping is False
+
+    adapter = make_mocked_engine_adapter(EngineAdapter)
+    assert adapter.with_settings()._pre_ping is False
+
+
 @pytest.mark.parametrize(
     "partitioned_by",
     [