Re: Proposed patch for showing originating query on error

On 28-Jul-05, at 11:06 AM, Oliver Jowett wrote:

> Dave Cramer wrote:
>
>
>> There are even more paths there to hook into. Putting it in
>> AbstractJdbc2Statement minimizes
>> the number of places it would need to be added to. QueryExecutor
>> has a
>> few Implementations ...
>>
>
> Well, do you have a better suggestion that catches all paths?
I'll give it some serious attention later, I hacked it in to your
code for someone, just wanted to get it into the discussion.
>
>
>>> - ideally it should be configurable (hook it into whatever came
>>> out of
>>> Kris' suggestions about including more info in server-generated
>>> exception, perhaps?)
>>>
>>
>> Well, lets get it working first before we make it configurable.
>>
>
> sigh :( let's get it right before we commit it at all.. in practise it
> seems "we'll fix this later" means "we won't fix it"
Agreed.
>
> -O
>
>