<div dir="auto">Maybe it’s not a big issue. </div><div dir="auto"><br></div><div dir="auto">But more than that the way postgresql using casting function is related to the procvolatile.</div><div dir="auto"><br></div><div dir="auto">I believe the solution for pgpool should be related to which casting function postgres using.</div><div dir="auto"><br></div><div dir="auto">In case the procvolatile for that cast function is “i” the query should be cached and if the procvolatile is “s” it shouldn’t.</div><div dir="auto"><br></div><div dir="auto">But maybe I am wrong.</div><div dir="auto"><br></div><div dir="auto">Do your magic.</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto"><br></div><div dir="auto">Avi</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 4 Jul 2022 at 14:36 Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp">ishii@sraoss.co.jp</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">> Hi,<br>
> <br>
> I found some issue with timestamptz cast.<br>
> See the following:<br>
> <br>
> 1. Select ‘2022-02-18 07:00:00.006547+02’::timestamptz;<br>
> <br>
> 2. Select ‘2022-02-18 07:00:00.006547+02’::timestamptz; ―> will retrieved<br>
> from cache<br>
> <br>
> 3. Set time zone to ‘Some time zone’;<br>
> <br>
> 4. Select ‘2022-02-18 07:00:00.006547+02’::timestamptz; ―> will returned<br>
> from cache but shouldn’t because the time zone has been changed.<br>
> <br>
> I think the right behaviour should be that if we using cast which involved<br>
> timezone like timestamptz or timetz these queries shouldn’t saved in cache.<br>
> <br>
> What are your thoughts?<br>
<br>
You are right. Let me think about how to deal with the case.<br>
<br>
> Thanks,<br>
> <br>
> Avi.<br>
> <br>
> On Mon, 4 Jul 2022 at 11:41 Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>> wrote:<br>
> <br>
>> Glad to hear that :-)<br>
>><br>
>> > My mistake it’s working like a charm :)<br>
>> ><br>
>> > On Mon, 4 Jul 2022 at 11:32 Avi Raboah <<a href="mailto:avi.raboah@gmail.com" target="_blank">avi.raboah@gmail.com</a>> wrote:<br>
>> ><br>
>> >> I added the patch and it still not working.<br>
>> >> After your change the query<br>
>> >> Select ‘2022-02-18 07:00:00.006547’::timestamp;<br>
>> >><br>
>> >> Still not cached<br>
>> >><br>
>> >> On Mon, 4 Jul 2022 at 11:21 Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>> wrote:<br>
>> >><br>
>> >>> Hi,<br>
>> >>><br>
>> >>> > Hi,<br>
>> >>> ><br>
>> >>> > I saw your patch thanks for that.<br>
>> >>><br>
>> >>> You are welcome.<br>
>> >>><br>
>> >>> > One question, in order to enable the not_used block you add, Do I<br>
>> need<br>
>> >>> to<br>
>> >>> > define this macro in the same page?<br>
>> >>><br>
>> >>> No. You should *not* define NOT_USED symbol. Otherwise, the block will<br>
>> >>> be enabled, which is opposite to what the patch wants to do.<br>
>> >>><br>
>> >>> > For example:<br>
>> >>> > #define NOT_USED<br>
>> >>> > #ifdef NOT_USED<br>
>> >>> > …<br>
>> >>> > …<br>
>> >>> > #endif<br>
>> >>> ><br>
>> >>> > Or I don’t need to add that ?<br>
>> >>> ><br>
>> >>> > Thanks,<br>
>> >>> ><br>
>> >>> > Avi.<br>
>> >>> ><br>
>> >>> > On Mon, 4 Jul 2022 at 9:54 Avi Raboah <<a href="mailto:avi.raboah@gmail.com" target="_blank">avi.raboah@gmail.com</a>> wrote:<br>
>> >>> ><br>
>> >>> >> Awesome, thanks!<br>
>> >>> >><br>
>> >>> >> On Mon, 4 Jul 2022 at 9:52 Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>> wrote:<br>
>> >>> >><br>
>> >>> >>> > Thanks a lot for your fast reaponse!<br>
>> >>> >>> ><br>
>> >>> >>> > Do you know when the fix will be available?<br>
>> >>> >>><br>
>> >>> >>> I have just pushed the fix. It will available in the next scheduled<br>
>> >>> >>> release (Aug 18).<br>
>> >>> >>><br>
>> >>> >>> <a href="https://pgpool.net/mediawiki/index.php/Roadmap" rel="noreferrer" target="_blank">https://pgpool.net/mediawiki/index.php/Roadmap</a><br>
>> >>> >>><br>
>> >>> >>> If you need patches, you can grab from the git repository.<br>
>> >>> >>><br>
>> >>> >>> <a href="https://pgpool.net/mediawiki/index.php/Source_code_repository" rel="noreferrer" target="_blank">https://pgpool.net/mediawiki/index.php/Source_code_repository</a><br>
>> >>> >>><br>
>> >>> >>> Best reagards,<br>
>> >>> >>> --<br>
>> >>> >>> Tatsuo Ishii<br>
>> >>> >>> SRA OSS, Inc. Japan<br>
>> >>> >>> English: <a href="http://www.sraoss.co.jp/index_en.php" rel="noreferrer" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
>> >>> >>> Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
>> >>> >>><br>
>> >>> >><br>
>> >>><br>
>> >><br>
>><br>
</blockquote></div></div>