Hi

Thank you for  the new version patch.
I have confirmed that the build succeeds and passes all regression tests on my 
machine.

```
  * Terminates all background workers connected to the specified database. 
However,
- * If the worker is interruptible.
+ * If the worker is interruptible. A signal is sent directly to the worker.
  */
```

This is a minor point,  sending a signal directly is to enable the bgworker 
restart setting, right?
I thought it would be helpful to clarify in the function description why a 
signal is sent directly.

For example, how about the following phrasing? Feel free to modify the wording 
as needed.

“A signal is sent directly to the worker so that it can be restarted according 
to the worker's bgw_restart_time setting.”

Regards,
Aya Iwata

Reply via email to