Re: [xmlblaster] Ldbc

[Peter Bennett <bencom at bencom.co.nz>]

Michele Laghi wrote:
> Hi Peter,
> I have looked at the code and in fact it should be possible to implement 
> all in one single common code by adding two more configuration parameters.
>
> Paring the mapping should be possible after some code modification (the 
> ',' inside a mapping should not break in future)
>
> Before I start however I want to make sure the current code works. I 
> tested the ldbc with postgres and there is one only error in the 
> testsuite. It fails when making the request in peekBySamePriority when 
> 12 entries in the queue and 4 entries expected to be peeked. The statement:
>
> "SELECT * from TEST_ENTRIES where 
> queueName='callback_QueuePluginpeekMsg' and nodeId='xmlBlaster' and 
> prio=(select max(prio) from TEST_ENTRIES where 
> queueName='callback_QueuePluginpeekMsg' AND nodeId='xmlBlaster')  ORDER 
> BY dataId ASC"
>
> with the error
>
> [Nov 18, 2003 10:05:08 PM TRACE LdbcPreparedQuery] Constructor. 
> SQLException: 37000
>
> Any idea about the cryptic 37000 ?
>
>
> About the tests: I think it is not necessary to write new tests: the 
> criteria that the old tests should pass should be sufficient.
>
> Saluti
> Michele

Hmmm strange... Postgres was the only db I was fully confident would 
work... I will look into the error... They are not that easy to track 
down. Thanks for the test.

As I said I am not sure I am erasing enough from the tables with the 
replacement code to overcome no cascading delete in ldbc however that 
may not explain the test results. Surely there are other tests that 
erase and check.

As for your other suggestion about integrating the code... I have no 
problem with this however we should wait for a while and get it stable 
and working first.

Regards