[pgpool-hackers: 4518] Re: Introducing Global Connection Pooling Feature for Pgpool-II

Tatsuo Ishii ishii at postgresql.org
Wed Aug 28 15:25:56 JST 2024


> Hi Usama,
> 
> I have looked into the GitHub repository you are working on for this
> project:
> 
> https://github.com/codeforall/pgpool2
> 
> I noticed that you modified src/main/main.c:
> 
> *** /home/t-ishii/work/Pgpool-II/current/pgpool2/src/main/main.c	2024-08-25 10:18:43.006798216 +0900
> --- src/main/main.c	2024-08-27 16:00:51.388409353 +0900
> ***************
> *** 314,320 ****
> --- 314,323 ----
>   	 * If ssl is enabled, initialize the SSL context
>   	 */
>   	if (pool_config->ssl)
> + 	{
>   		SSL_ServerSide_init();
> + 		SSL_ClientSide_init();
> + 	}
>   #endif							/* USE_SSL */
>   
>   	/* check effective user id for watchdog */
> 
> Can you explain why you need the modification? I thought SSL is not
> related to the global connection feature...

Probably I found the answer myself. It seems the commit message
explains the reason of the change.

https://github.com/pgpool/pgpool2/commit/9edbb207840704c35b4e405f067c1323ed13c400

So I ran the regression test and got a few failures. Among them I have
fixed 010.rewrite_timestamp. Patch attached.

Best reagards,
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix-010.patch
Type: text/x-patch
Size: 870 bytes
Desc: not available
URL: <http://www.pgpool.net/pipermail/pgpool-hackers/attachments/20240828/8ac536a4/attachment.bin>


More information about the pgpool-hackers mailing list