Do not reload config file on PS reconnect

[knizhnik]

Problem

See #10184
and
https://neondb.slack.com/archives/C04DGM6SMTM/p1733997259898819

Reloading config file inside parallel worker cause it's termination

Summary of changes

Remove call of HandleMainLoopInterrupts()
Update of page server URL is propagated by postmaster through shared memory and we should not reload config for it.

[MMeent]

LGTM, but could use more descriptive description - just referencing the issue is a bit too little IMO.

[hlinnaka]

Hmm, so the HandleMainLoopinterrupts() call was added in commit b989ad1. The same commit added the mechanism where the connection string is kept in shared memory, and updated by postmaster (or any other process). ISTM that the HandleMainLoopinterrupts() call was never necessary, the ReloadConnstring() (later replaced with load_shard_map) that was added to pageserver_connect was sufficient for backends to have the latest connection string. Commit b989ad1 included a regression test, so if that still passes, we're good.

[github-actions]

7095 tests run: 6797 passed, 0 failed, 298 skipped (full report)

---

Flaky tests (4)

Postgres 17

• test_physical_replication_config_mismatch_too_many_known_xids: release-arm64
• test_sharding_split_failures[failure0]: debug-x86-64

Postgres 16

• test_lr_with_slow_safekeeper: release-arm64

Postgres 14

• test_timeline_archive[4]: release-arm64

Code coverage* (full report)

• functions: 31.3% (8397 of 26870 functions)
• lines: 48.0% (66639 of 138946 lines)

* collected from Rust tests only

---

_{The comment gets automatically updated with the latest test results
799c4fc at 2024-12-19T14:59:56.659Z :recycle:}