Greg Sabino Mullane wrote:
> > The current way is ok for me at the moment. I still think there is a better
> > way (parsing statements like it's already done for
> > no-transaction-allowed-statements), but hey, as soon as your patch will be
> > applied, I can myself propose another patch to improve this. ;-)
>   
> Parsing the statment will not help: even if the statement is a savepoint, we
> need to wrap it in case we need to roll it back. The only other option I
> can see to my patch is to, upon a successful user savepoint creation,
> roll back their savepoint and immediately reissue it. That seems worse to
> me than having N*2 savepoints though.

Agreed.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 7: don't forget to increase your free space map settings

Reply via email to