<div dir="ltr"><div><p>Hi Ishii-san,</p><p>Thank you for reviewing the global connection pool implementation and for the patch to fix the rewrite timestamp test case. I have pushed your changes to the GitHub repository.</p><p>The SSL context was created before spawning child processes, which was necessary because the global connection pool needs to share backend sockets across different processes, which requires a consistent SSL context in all child processes. There is still a TODO item on my list to pass the SSL session back to the global connection pool along with the socket when the backend uses SSL. I am working on finding some time to implement this.</p><p>Additionally, I have fixed the connection idle time functionality, so we should see one less failure in the regression tests. I am continuing to work on the missing functionality to ensure the regression tests pass cleanly.</p></div><div><br></div><div>Thanks</div><div>Best regards</div><div>Muhammad Usama</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 28, 2024 at 11:27 AM Tatsuo Ishii <<a href="mailto:ishii@postgresql.org">ishii@postgresql.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Hi Usama,<br>
> <br>
> I have looked into the GitHub repository you are working on for this<br>
> project:<br>
> <br>
> <a href="https://url.avanan.click/v2/___https://github.com/codeforall/pgpool2___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6MTljNjpmMDRhZGQ5YWZiNjU3MThhZTBhY2U1NzJjODQwYTE3N2RkNmRmOTBjMmE5ZjRlMmRlMDU5MjBjOGZmNzgzNmE2OnA6VDpO" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___https://github.com/codeforall/pgpool2___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6MTljNjpmMDRhZGQ5YWZiNjU3MThhZTBhY2U1NzJjODQwYTE3N2RkNmRmOTBjMmE5ZjRlMmRlMDU5MjBjOGZmNzgzNmE2OnA6VDpO</a><br>
> <br>
> I noticed that you modified src/main/main.c:<br>
> <br>
> *** /home/t-ishii/work/Pgpool-II/current/pgpool2/src/main/main.c      2024-08-25 10:18:43.006798216 +0900<br>
> --- src/main/main.c   2024-08-27 16:00:51.388409353 +0900<br>
> ***************<br>
> *** 314,320 ****<br>
> --- 314,323 ----<br>
>        * If ssl is enabled, initialize the SSL context<br>
>        */<br>
>       if (pool_config->ssl)<br>
> +     {<br>
>               SSL_ServerSide_init();<br>
> +             SSL_ClientSide_init();<br>
> +     }<br>
>   #endif                                                      /* USE_SSL */<br>
>   <br>
>       /* check effective user id for watchdog */<br>
> <br>
> Can you explain why you need the modification? I thought SSL is not<br>
> related to the global connection feature...<br>
<br>
Probably I found the answer myself. It seems the commit message<br>
explains the reason of the change.<br>
<br>
<a href="https://url.avanan.click/v2/___https://github.com/pgpool/pgpool2/commit/9edbb207840704c35b4e405f067c1323ed13c400___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6YTgxNDo3NWI4ZWVhYjRmM2QwZDllZjlkNTE3NmE3M2RmMjNhYmIyMmQ0Njc1MmEyMGQxNmEzZGFjYmYzNTg4NGUxMDRiOnA6VDpO" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___https://github.com/pgpool/pgpool2/commit/9edbb207840704c35b4e405f067c1323ed13c400___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6YTgxNDo3NWI4ZWVhYjRmM2QwZDllZjlkNTE3NmE3M2RmMjNhYmIyMmQ0Njc1MmEyMGQxNmEzZGFjYmYzNTg4NGUxMDRiOnA6VDpO</a><br>
<br>
So I ran the regression test and got a few failures. Among them I have<br>
fixed 010.rewrite_timestamp. Patch attached.<br>
<br>
Best reagards,<br>
--<br>
Tatsuo Ishii<br>
SRA OSS K.K.<br>
English: <a href="https://url.avanan.click/v2/___http://www.sraoss.co.jp/index_en/___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6ZjA0NTo5ZmYyNWFjZjA5M2RjMjBkZWU2MDczNTBjMzM3YzE1MTY1NzBiZDNiOGFmM2JiNTYzNThhNjBmZjcxOTBlZDhhOnA6VDpO" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___http://www.sraoss.co.jp/index_en/___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6ZjA0NTo5ZmYyNWFjZjA5M2RjMjBkZWU2MDczNTBjMzM3YzE1MTY1NzBiZDNiOGFmM2JiNTYzNThhNjBmZjcxOTBlZDhhOnA6VDpO</a><br>
Japanese:<a href="https://url.avanan.click/v2/___http://www.sraoss.co.jp___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6NGRiNjpjYzljODJlYTRjZjM1MmI1MzllYzllNWU3NzI3ZjhkNzlhYzdkODc2ODhlZTRjNTliMzE1YWZhM2U1NmZiN2UzOnA6VDpO" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___http://www.sraoss.co.jp___.YXAzOnBlcmNvbmE6YTpnOmQ2NTFkYmFkZjVkNTM3ZDc5YWRiMzQ0YTAxZjEzYzc5OjY6NGRiNjpjYzljODJlYTRjZjM1MmI1MzllYzllNWU3NzI3ZjhkNzlhYzdkODc2ODhlZTRjNTliMzE1YWZhM2U1NmZiN2UzOnA6VDpO</a><br>
</blockquote></div>