----- Original Message -----
From: "Serge Knystautas" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Tuesday, October 09, 2001 11:18 AM
Subject: Re: James with SAPDB


> Thanks for your patch.  I have some questions though...
>
> I'm looking at sqlResources.xml, and I see you changed the createTable
> procedure, but didn't use the db="sapdb".  Something like this should work
> as the JDBC driver will clue sql resources into knowing that the db is
> "sapdb", and thus use this overridden createTable sql...
>
>     <sql name="createTable" db="sapdb">
>         CREATE TABLE ${table} (
>          message_name varchar (200) NOT NULL,
>          repository_name varchar (200) NOT NULL,
>          message_state varchar (30) NOT NULL ,
>          error_message varchar (200) NULL ,
>          sender varchar (200) NULL ,
>          recipients long NOT NULL ,
>          remote_host varchar (100) NOT NULL ,
>          remote_addr varchar (20) NOT NULL ,
>          message_body long NOT NULL ,
>          last_updated date NOT NULL,
>          PRIMARY KEY (message_name, repository_name)
>         )
>     </sql>
>
> Does that look correct for the SAPDB?

That is correct. I think your version is better
for the common good :)

>
> Also, what about this in JDBCMailStore...
> file://insertMessage.setBytes(10, headerOut.toByteArray());
>
> insertMessage.setString(10, headerOut.toString());
>
> Is there a reason you did toString instead of toByteArray?  I'm not sure
how
> we could support that with the way the code is designed.  And in
> MimeMessageJDBCSource you retrieve the header as a String.  Does the SAPDB
> JDBC driver not support binary data?
>
> I'd like to integrate the code, but our design can't currently incorporate
> the String <-> byte[] difference.

Actually I have to patch James because there is a bug
in SAPDB jdbc driver. Actually I am considering to patch
SAPDB jdbc driver however James code is much cleaner
so it is easier for me to patch James :) Please do not integrate
my code. James code is more correct I just submit the patch
because I think it might help somebody that struggle with the
same problem.

Thanks for your response.
>
> Serge Knystautas
> Loki Technologies
> http://www.lokitech.com/
> ----- Original Message -----
> From: "Samuel Franklyn" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Thursday, October 04, 2001 8:46 AM
> Subject: James with SAPDB
>
>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to