svn commit: r1158521 - in /james/mailbox/trunk/store/src/main/java/org/apache/james/mailbox/store: MimeDescriptorImpl.java ResultUtils.java StoreMessageManager.java search/MessageSearcher.java streami

2011-08-16 Thread norman
Author: norman Date: Wed Aug 17 05:30:04 2011 New Revision: 1158521 URL: http://svn.apache.org/viewvc?rev=1158521&view=rev Log: Don't force a specifc header length. This should get done by the protocol (SMTP/IMAP etc) not the mailbox impl Modified: james/mailbox/trunk/store/src/main/java/or

Re: [VOTE] Release Apache James 3.0-beta3

2011-08-16 Thread Patrick Pyette
Based on Robert's email, if anyone can cast a vote, then I shall :-) > [X] +1 Release Apache James 3.0-beta3 > [ ] +0 Tending towards acceptance (without time for full review) > [ ] -0 Tending towards rejection > [ ] -1 Do not release Apache James 3.0-beta3 (with reasons, please) Thanks for all o

svn commit: r1158435 - /james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/ImapChannelUpstreamHandler.java

2011-08-16 Thread norman
Author: norman Date: Tue Aug 16 20:24:23 2011 New Revision: 1158435 URL: http://svn.apache.org/viewvc?rev=1158435&view=rev Log: Flush the hello response to the composer Modified: james/server/trunk/imapserver/src/main/java/org/apache/james/imapserver/netty/ImapChannelUpstreamHandler.java Mo

R: [VOTE] Release Apache James 3.0-beta3

2011-08-16 Thread Vincenzo Gianferrari Pini
[X] +1 Release Apache James 3.0-beta3 Great job Robert! Vincenzo Inviato dal dispositivo wireless BlackBerry® -Original Message- From: Robert Burrell Donkin Date: Mon, 15 Aug 2011 11:33:32 To: James Developers List Reply-To: "James Developers List" Subject: [VOTE] Release Apache James

Rat Whisker [WAS Re: [VOTE] Release Apache James 3.0-beta3]

2011-08-16 Thread Robert Burrell Donkin
On Tue, Aug 16, 2011 at 9:15 AM, Eric Charles wrote: > [x] +1 Release Apache James 3.0-beta3 > > IMAP4, POP3, SMTP working fine. > > Thx Robert for the NOTICE. Is it fully RAT automated or still manual updates > needed? The validation and generation functions are working from the command line. To

Re: NoSQL backend for James

2011-08-16 Thread Pepijn de Vos
I think the link has gone missing, but it would indeed be very helpful if you wrote a bit about your experience navigating James. > And nice to see you are interested. I worked on the HBase mailbox > implementation so I can help also. I am preparing a schema design [1] -

Re: mailbox integration tests preview

2011-08-16 Thread Ioan Eugen Stan
2011/8/16 Norman Maurer : > Am 16.08.2011 11:25, schrieb Ioan Eugen Stan: >> >> 2011/8/16 Eric Charles: >>> >>> Hi Ioan, >>> >>> I've run a few tests but didn't see the ClassCastException. >> >> Thats because last night some gremlins worked on the code and >> committed the fixes for that ;). >> >>>

Re: mailbox integration tests preview

2011-08-16 Thread Norman Maurer
Am 16.08.2011 11:25, schrieb Ioan Eugen Stan: 2011/8/16 Eric Charles: Hi Ioan, I've run a few tests but didn't see the ClassCastException. Thats because last night some gremlins worked on the code and committed the fixes for that ;). e.g. Running from eclipse: NonAuthenticatedStateTest : 10

Re: mailbox integration tests preview

2011-08-16 Thread Ioan Eugen Stan
2011/8/16 Eric Charles : > Hi Ioan, > > I've run a few tests but didn't see the ClassCastException. Thats because last night some gremlins worked on the code and committed the fixes for that ;). > > e.g. Running from eclipse: > > NonAuthenticatedStateTest : 100% OK > > SelectTest : > org.apache.j

Re: mailbox integration tests preview

2011-08-16 Thread Ioan Eugen Stan
2011/8/16 Norman Maurer : > Hi Ioan, > > I just found a bug in your impl. Here is the fix.. > > diff -r edb0b99712ce > src/main/java/org/apache/james/mailbox/hbase/mail/HBaseMessageMapper.java > --- > a/src/main/java/org/apache/james/mailbox/hbase/mail/HBaseMessageMapper.java >    Tue Aug 16 03:07:

Re: mailbox integration tests preview

2011-08-16 Thread Eric Charles
Hi Ioan, I've run a few tests but didn't see the ClassCastException. e.g. Running from eclipse: NonAuthenticatedStateTest : 100% OK SelectTest : org.apache.james.imap.tester.base.ProtocolSession$InvalidServerResponseException: Location: /org/apache/james/imap/scripts/SelectUnseen.test:24 L

svn commit: r1158154 - /james/server/trunk/hbase/src/main/java/org/apache/james/system/hbase/TablePool.java

2011-08-16 Thread eric
Author: eric Date: Tue Aug 16 08:40:45 2011 New Revision: 1158154 URL: http://svn.apache.org/viewvc?rev=1158154&view=rev Log: Add synchronized to ensure thread-safety on singleton (JAMES-1273) Modified: james/server/trunk/hbase/src/main/java/org/apache/james/system/hbase/TablePool.java Modi

Re: [GSOC] Hbase Mailbox code review

2011-08-16 Thread Eric Charles
Hi, Yes, very good work Ioan :) +1 on Norman comments (sounds like 'Less is More'). I guess you are focusing on integration tests for the final rush (deadline is approaching). We could review the tablepool, the indexes on datamodel... later after gsoc. Thx. On 16/08/11 09:26, Norman Maurer wr

Re: [VOTE] Release Apache James 3.0-beta3

2011-08-16 Thread Eric Charles
[x] +1 Release Apache James 3.0-beta3 IMAP4, POP3, SMTP working fine. Thx Robert for the NOTICE. Is it fully RAT automated or still manual updates needed? Eric On 15/08/11 12:33, Robert Burrell Donkin wrote: It's been a long (apologies) and rocky (dev box MIA) road but I hope we're finally

Re: mailbox integration tests preview

2011-08-16 Thread Norman Maurer
Hi Ioan, I just found a bug in your impl. Here is the fix.. diff -r edb0b99712ce src/main/java/org/apache/james/mailbox/hbase/mail/HBaseMessageMapper.java --- a/src/main/java/org/apache/james/mailbox/hbase/mail/HBaseMessageMapper.java Tue Aug 16 03:07:49 2011 +0300 +++ b/src/main/java/org

[GSOC] Hbase Mailbox code review

2011-08-16 Thread Norman Maurer
Hi Ioan, I did take the time to review your code and It seems to be in a very good shape :) I just found some minor things that should porlly get addressed: * Merge AbstractHBaseMessage with HBaseMessage ( I see no point to have them seperated) * Move the merged version to the package org.a