| From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
|---|---|
| To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: extend pgbench expressions with functions |
| Date: | 2016-01-28 09:31:42 |
| Message-ID: | alpine.DEB.2.10.1601281016400.27415@sto |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
>> I do not think that it is really worth fixing, but I will not prevent anyone
>> to fix it.
>
> I still think it does. Well, if there is consensus to address this one
> and optionally the other integer overflows even on back branches, I'll
> write a patch and let's call that a deal. This is not a problem from
> my side.
My point is just about the cost-benefit of fixing a low probability issue
that you can only encounter if you are looking for it, and not with any
reasonable bench script.
Now adding somewhere a test might just help closing the subject and
do more useful things, so that would also be a win.
/* these would raise an arithmetic error */
if (lval == INT64_MIN && rval == -1)
{
fprintf(stderr, "cannot divide or modulo INT64_MIN by -1\n");
return false;
}
This may be backpatched to old supported versions.
--
Fabien.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2016-01-28 10:06:24 | Re: Template for commit messages |
| Previous Message | Alvaro Herrera | 2016-01-28 09:15:28 | Re: Minor code improvements to create_foreignscan_plan/ExecInitForeignScan |