Thanks for the notes. I'll store the patch for the SQL script and put some notes somewhere about the necessary class differences because of the jdbc driver.
Serge Knystautas Loki Technologies http://www.lokitech.com/ ----- Original Message ----- From: "Samuel Franklyn" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, October 09, 2001 10:41 AM Subject: Re: James with SAPDB > ----- 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] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
