Bruce Momjian wrote:
> I liked option #2. I don't think the _last_ query in a rule should have
> any special handling.
>
> So, to summarize #2, we have:
>
> if no INSTEAD,
> return value of original command
>
> if INSTEAD,
> return tag of original command
> return sum of all affected rows with the same tag
> return OID if all INSERTs in the rule insert only one row, else zero
>
How about:
if no INSTEAD, return value of original command
if INSTEAD, return tag MUTATED return sum of sum of tuple counts of all replacement commands return OID if
sumof all replacement INSERTs in the rule inserted only one row, else zero
This is basically Tom's proposal, but substituting MUTATED for the
original command tag name acknowledges that the original command was not executed unchanged. It also serves as a
warningthat the affected
tuple count is from one or more substitute operations, not the original
command.
> I don't think we should add tuple counts from different commands, i.e.
> adding UPDATE and DELETE counts just yeilds a totally meaningless
> number.
I don't know about that. The number of "rows affected" is indeed this
number. It's just that they were not all affected in the same way.
> I don't think there is any need/desire to add additional API routines to
> handle multiple return values.
Agreed.
>
> Can I get some votes on this? We have one user very determined to get a
> fix, and the TODO.detail file has another user who really wants a fix.
+1 for the version above ;-)
Joe
