- Notifications
You must be signed in to change notification settings - Fork5
Commitb5dd50f
committed
Rewrite async-connection loop in libpqwalreceiver.c, once again.
The original coding in commit1e8a850 didn't use PQconnectPoll perspec, and while the rewrite ine434ad3 is closer, it still doesn'tguarantee to wait until the socket is read-ready or write-ready (asappropriate) before calling PQconnectPoll. It's not clear whetherthat omission is causing the continuing failures on buildfarm memberbowerbird; but given the lack of other explanations meeting theavailable facts, let's tighten that up and see what happens.An independent issue in the same loop was that it had a race conditionwhereby it could clear the process's latch without having serviced aninterrupt request, causing failure to respond to a cancel while waitingfor connection (the very problem1e8a850 was meant to fix).Discussion:https://postgr.es/m/7295.1489596949@sss.pgh.pa.us1 parent1ea60ad commitb5dd50f
File tree
1 file changed
+30
-30
lines changed- src/backend/replication/libpqwalreceiver
1 file changed
+30
-30
lines changedLines changed: 30 additions & 30 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
159 | 159 |
| |
160 | 160 |
| |
161 | 161 |
| |
162 |
| - | |
163 |
| - | |
| 162 | + | |
164 | 163 |
| |
165 |
| - | |
166 |
| - | |
167 |
| - | |
| 164 | + | |
| 165 | + | |
168 | 166 |
| |
169 |
| - | |
170 |
| - | |
171 |
| - | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
172 | 188 |
| |
173 |
| - | |
174 |
| - | |
175 |
| - | |
176 |
| - | |
177 |
| - | |
178 |
| - | |
179 |
| - | |
180 | 189 |
| |
181 |
| - | |
182 |
| - | |
183 |
| - | |
184 |
| - | |
185 |
| - | |
186 |
| - | |
187 |
| - | |
188 |
| - | |
189 |
| - | |
190 |
| - | |
191 |
| - | |
192 |
| - | |
193 |
| - | |
194 |
| - | |
| 190 | + | |
195 | 191 |
| |
196 |
| - | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
197 | 197 |
| |
198 | 198 |
| |
199 | 199 |
| |
|
0 commit comments
Comments
(0)