Uniforms the errors msgs at tuplestore paths
Hi,
Like how the commit
https://github.com/postgres/postgres/commit/07daca53bfcad59618a9c6fad304e380cc9d2bc1
The are some paths that were missed:
-At jsonfuncs.c
The errors mgs do not report about the materialize mode.
-At plperl.c
The function plperl_return_next_internal does not check rsi appropriately.
The error about materialize mode required, is missed.
-At pl_exec.c
The error mgs do not report about the materialize mode
-At pltcl.c:
The function pltcl_init_tuple_store does not check rsi appropriately.
Patch attached.
regards,
Ranier Vilela
Attachments:
v1-0001-uniform-error-msgs-tuplestore.patchapplication/octet-stream; name=v1-0001-uniform-error-msgs-tuplestore.patchDownload+56-21
On Sun, Feb 20, 2022 at 11:12:42AM -0300, Ranier Vilela wrote:
Like how the commit
https://github.com/postgres/postgres/commit/07daca53bfcad59618a9c6fad304e380cc9d2bc1
The are some paths that were missed:
I think these are all unified by the existing tuplestore patch.
https://commitfest.postgresql.org/37/3420/
Show quoted text
-At jsonfuncs.c
The errors mgs do not report about the materialize mode.
-At plperl.c
The function plperl_return_next_internal does not check rsi appropriately.
The error about materialize mode required, is missed.
-At pl_exec.c
The error mgs do not report about the materialize mode
-At pltcl.c:
The function pltcl_init_tuple_store does not check rsi appropriately.
Em dom., 20 de fev. de 2022 às 11:30, Justin Pryzby <pryzby@telsasoft.com>
escreveu:
On Sun, Feb 20, 2022 at 11:12:42AM -0300, Ranier Vilela wrote:
Like how the commit
https://github.com/postgres/postgres/commit/07daca53bfcad59618a9c6fad304e380cc9d2bc1
The are some paths that were missed:
I think these are all unified by the existing tuplestore patch.
https://commitfest.postgresql.org/37/3420/
I can't see:
plperl.c
pl_exec.c
pttcl.c
Only jsonfuncs.c, but the error about "materialized mode" is not reported.
regards,
Ranier Vilela
On Sun, Feb 20, 2022 at 11:37:33AM -0300, Ranier Vilela wrote:
I can't see:
plperl.c
pl_exec.c
pttcl.cOnly jsonfuncs.c, but the error about "materialized mode" is not reported.
Melanie has done a nice analysis of all the code paths doing
materialization checks for her patch with SRF functions. Though there
are parts that can be simplified to reduce the differences in check
patterns before doing the overall refactoring, I think that we'd
better keep any discussion related to this topic on the other thread
rather than splitting the effort across more threads.
--
Michael
Em dom., 20 de fev. de 2022 às 22:08, Michael Paquier <michael@paquier.xyz>
escreveu:
On Sun, Feb 20, 2022 at 11:37:33AM -0300, Ranier Vilela wrote:
I can't see:
plperl.c
pl_exec.c
pttcl.cOnly jsonfuncs.c, but the error about "materialized mode" is not
reported.
Melanie has done a nice analysis of all the code paths doing
materialization checks for her patch with SRF functions. Though there
are parts that can be simplified to reduce the differences in check
patterns before doing the overall refactoring, I think that we'd
better keep any discussion related to this topic on the other thread
rather than splitting the effort across more threads.
Fine, I agree.
regards,
Ranier Vilela
Em dom., 20 de fev. de 2022 às 22:45, Ranier Vilela <ranier.vf@gmail.com>
escreveu:
Em dom., 20 de fev. de 2022 às 22:08, Michael Paquier <michael@paquier.xyz>
escreveu:On Sun, Feb 20, 2022 at 11:37:33AM -0300, Ranier Vilela wrote:
I can't see:
plperl.c
pl_exec.c
pttcl.cOnly jsonfuncs.c, but the error about "materialized mode" is not
reported.
Melanie has done a nice analysis of all the code paths doing
materialization checks for her patch with SRF functions. Though there
are parts that can be simplified to reduce the differences in check
patterns before doing the overall refactoring, I think that we'd
better keep any discussion related to this topic on the other thread
rather than splitting the effort across more threads.Fine, I agree.
Thanks for the commit Michael.
regards,
Ranier Vilela