Re: extend pgbench expressions with functions
От | Fabien COELHO |
---|---|
Тема | Re: extend pgbench expressions with functions |
Дата | |
Msg-id | alpine.DEB.2.10.1601131417250.2188@sto обсуждение исходный текст |
Ответ на | Re: extend pgbench expressions with functions (Michael Paquier <michael.paquier@gmail.com>) |
Ответы |
Re: extend pgbench expressions with functions
|
Список | pgsql-hackers |
Hello Michaël, >> ISTM that if pgbench is to be stopped, the simplest option is just to abort >> with a nicer error message from the get*Rand function, there is no need to >> change the function signature and transfer the error management upwards. > > That's fine to me, as long as the solution is elegant. Hmmm, this is subjective:-) I've decided to stay with the current behavior (\setrandom), that is to abort the current transaction on errors but not to abort pgbench itself. The check is done before calling the functions, and I let an assert in the functions just to be sure. It is going to loop on errors anyway, but this is what it already does anyway. >> Ok. I'm hesitating about removing the operator management, especially if I'm >> told to put it back afterwards. > > I can understand that, things like that happen all the time here and > that's not a straight-forward patch that we have here. I am sure that > additional opinions here would be good to have before taking one > decision or another. With the current statu-quo, let's just do what > you think is best. I let the operators alone and just adds functions management next to it. I'll merge operators as functions only if it is a blocker. I have assumed that your v4 is really v17, and this is v18... -- Fabien.
В списке pgsql-hackers по дате отправления: