Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make async connect work on MacOS: it is necessary top call WaitLatchOrSocket before PQconnectPoll #8472

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

knizhnik
Copy link
Contributor

Problem

While investigating problem with test_subscriber_restart flukyness, I found out that this test is not passed at all for PG 14/15 at MacOS (while working for PG16).

Summary of changes

Rewrite async connect state machine exactly in the same way as in Vanilla: call WaitLatchOrSocket with WL_SOCKETR_WRTEABLE before calling PQconnectPoll.

Please notice that most likely it will not fix flukyness of test_subscriber_restart.

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
  • If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

  • Do not forget to reformat commit message to not include the above checklist

@knizhnik knizhnik requested review from a team as code owners July 23, 2024 16:13
@petuhovskiy petuhovskiy requested review from arssher and removed request for petuhovskiy July 23, 2024 16:14
Copy link

github-actions bot commented Jul 23, 2024

3276 tests run: 3155 passed, 0 failed, 121 skipped (full report)


Flaky tests (5)

Postgres 16

  • test_sharding_split_compaction[compact-shard-ancestors-localonly]: debug
  • test_change_pageserver: debug

Postgres 15

  • test_change_pageserver: debug
  • test_cancellations: debug

Postgres 14

  • test_scrubber_physical_gc_ancestors[2]: debug

Code coverage* (full report)

  • functions: 32.6% (7005 of 21497 functions)
  • lines: 50.0% (55259 of 110586 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
98743ea at 2024-07-24T05:50:30.413Z :recycle:

@knizhnik knizhnik merged commit 925c5ad into main Jul 24, 2024
65 of 69 checks passed
@knizhnik knizhnik deleted the async_connect branch July 24, 2024 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants