On 4/6/06, Gianny Damour <gianny.damour@optusnet.com.au> wrote: Hi Matt,

In trunk, we already have the individual update statement s= trategy
turned on by default. If you want to switch back to the universa= l update
command, i.e the "CASE THEN" approach, then you can f= lag an entity with
the static-sql element.

So, I think that we are ok.

Thank= s,
Gianny

Matt Hogstrom wrote:

> Gianny,
>
>= ; I think we should consider changing the default behaviour for "SET
> <column> =3D CASE WHEN ? THEN ?
> > ELSE <column= >".  DB2 doesn't like it, Oracle doesn't like it, Derby> doesn't like it and Firebird apparently doesn't like it :)
>> I think I'd be in favor of a universal update command that updates a= ll
> columns regardless if they have changed or individual update
&g= t; statements for only the columns that have changed.
>
> What = are your thoughts?
>
> Matt
>
> Gianny Damour wrote= :
>
>> Hi Olivier,
>>
>> I think that Fireb= ird does not like the generated UPDATE statements.
>> More accurat= ely it does not like the "SET <column> =3D CASE WHEN ? THEN
>> ? ELSE <column>" syntax.
>>
>> This p= roblem has been fixed in head as the UPDATE statements will
>> onl= y use the "SET <column> =3D ?" syntax. Meanwhile, do you kn= ow what
>> the Firebird syntax is to achieve the same result than "S= ET <column>
>> =3D CASE WHEN ? THEN ? ELSE <column>&qu= ot;. If you give us this syntax, I
>> think that it should be quic= k easy to fix this problem (basically, we
>> can implement a DBSyntaxFactory and enable it by setting the>> db-syntax-factory element of the openejb-jar DD).
>>>> Thanks,
>> Gianny
>>
>> Olivier Voutat= wrote:
>>
>>> Lol, there is no space, it is just the Ctrl-c = Ctrl-v, cause there
>>> was no space in my writing e-mail windo= w so it did a return line.
>>>
>>> Anyway, maybe it= is a tip of which is the problem, but I tried to
>>> use my findBySituation, and it works BUT I don't get to al= ter the
>>> LocalCar (s)  returned.
>>>>>> My code in my SessionBean
>>
>>
>>= ;
>> <snip>
>>
>>>
>>> 00:34:32,073 WARN  [= GeronimoConnectionEventListener]
>>> connectionErrorOccurred ca= lled with null
>>> org.firebirdsql.jdbc.FBSQLException: GDS Exc= eption. 335544569.
>>> Dynamic SQL Error
>>> SQL error code =3D -104<= br>>>> Token unknown - line 1, char 39
>>> THEN
>= ;>>         at
>>>= ; org.firebirdsql.jdbc.AbstractPreparedStatement .<init>(AbstractPreparedStatement.java:118)
>>>
>&g= t;>         at org.firebirdsql.j= dbc.FBPreparedStatement
>>> .<init>(FBPreparedStatement.j= ava:40)
>>
>>
>>
>> <snip>
>>
>>> Caused by: org.tranql= .ql.QueryException: Error executing statement:
>>> UPDATE CARRO= S SET MARCA =3D CASE WHEN ? THEN ? ELSE MARCA END, MODELO
>>> = =3D CASE WHEN ? THEN ? ELSE MODELO END, PLACA =3D CASE WHEN ? THEN ?
>>> ELSE PLACA END, QUILOMETRAGEM =3D CASE WHEN ? THEN ? ELSE<= br>>>> QUILOMETRAGEM END, COR =3D CASE WHEN ? THEN ? ELSE COR END,= SITUACAO =3D
>>> CASE WHEN ? THEN ? ELSE SITUACAO END, TIPO = =3D CASE WHEN ? THEN ? ELSE
>>> TIPO END, DIARIA =3D CASE WHEN ? THEN ? ELSE DIARIA END WH= ERE CHASSI =3D ?
>>>       &= nbsp; at
>>> org.tranql.sql.jdbc.JDBCUpdateCommand.execute(JDBC= UpdateCommand.java:69)
>>>
>>>         at
>>= > org.tranql.cache.SimpleFlushStrategy.flush(SimpleFlushStrategy.java:64= )
>>>         at org.tr= anql.cache.SimpleFlushStrategy.flush
>>> (SimpleFlushStrategy.j= ava :49)
>>>         at
= >>> org.tranql.cache.cache.InTxCacheTracker.flush(InTxCacheTracker= .java:41)
>>>         a= t org.tranql.cache.InTxCache.flush(InTxCache.java:86)
>>> =         at
>>> org.apache.geronimo.transaction.context.AbstractTransactio= nContext.flushState
>>> (AbstractTransactionContext.java:115)>>>         at
>&g= t;> org.apache.geronimo.transaction.context.InheritableTransactionContex= t.complete (InheritableTransactionContext.java:175)
>>>
>>>&nb= sp;        ... 24 more
>>> C= aused by: org.firebirdsql.jdbc.FBSQLExcep tion : GDS Exception.
>>= > 335544569. Dynamic SQL Error
>>> SQL error code =3D -104
>>> Token unknown - line 1= , char 39
>>> THEN
>>>     = ;    at
>>> org.firebirdsql.jdbc.AbstractPrepare= dStatement.<init>(AbstractPreparedStatement.java :118)
>>>
>>
>>
>>
>>
&g= t;>
>>
>>
>>
>
>