agora inbox for pljava-dev@postgresql.org  

help / color / mirror / Atom feed

Subject: [Pljava-dev] Cancelling trigger operation
Date: Mon, 23 Oct 2006 11:12:12 +0200
Message-ID: <453C876C.1050506@tada.se> (raw)
In-Reply-To: <453C82E7.7050700@logix-tt.com>
References: <449f4cf70610180438x3c928febw693667dae457e841@mail.gmail.com>
	<45367CD0.8090202@tada.se>
	<003601c6f679$9e1af3c0$1e03a8c0@fhrbek>
	<453C8163.8090100@tada.se>
	<453C82E7.7050700@logix-tt.com>

I see what you mean :-) and yes, I do think it is a bit too cryptic 
since it suggests that there actually is a row that can be deleted. It 
might lead to some semantic confusion. The reason the new and old row is 
represented as ResultSet instances are not that they are sets as such, 
it's just the best interface when dealing with a tuple. I don't want to 
add to that confusion by allowing the deletion of the tuple that they 
represent. An explicit method that denotes the cancellation of the 
trigger is a better choice, IMO.

Regards,
Thomas Hallgren

Markus Schaber wrote:
> Hi, Thomas,
>
> Thomas Hallgren wrote:
>
>   
>> Something is definitely missing in the TriggerData interface. It should 
>> have a 'setCancelled()' method that causes the returned tuple to be 
>> null. Please add a bug for this.
>>     
>
> What about using TriggerData.getNew().deleteRow() - is this too cryptic?
>
> Thanks,
> Markus
>
>   

view thread (6+ messages)  latest in thread

Message-ID: <453C876C.1050506@tada.se>
Permalink:  ../453C876C.1050506@tada.se/
Also on:    postgresql.org/message-id/453C876C.1050506@tada.se

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pljava-dev@postgresql.org
  Subject: Re: [Pljava-dev] Cancelling trigger operation
  In-Reply-To: <453C876C.1050506@tada.se>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox