Re: hbase-mailbox alpha release

2011-08-19 Thread Robert Burrell Donkin
On Fri, Aug 19, 2011 at 2:20 PM, Ioan Eugen Stan wrote: > Hello again, > > I ran the whole test suite again, to test my new 4GB RAM that I got > just for this. Currently the tests are looking like this: > > Tests run: 291, Failures: 0, Errors: 113, Skipped: 0 > That's a little over 61% test pass.

Re: hbase-mailbox alpha release

2011-08-19 Thread Ioan Eugen Stan
Hello again, I ran the whole test suite again, to test my new 4GB RAM that I got just for this. Currently the tests are looking like this: Tests run: 291, Failures: 0, Errors: 113, Skipped: 0 That's a little over 61% test pass. I expect to pass all tests by tonight. Code is not committed yet. By

Re: hbase-mailbox alpha release

2011-08-17 Thread Ioan Eugen Stan
2011/8/17 Eric Charles : > On 17/08/11 18:39, Ioan Eugen Stan wrote: >> >> 2011/8/17 Eric Charles: >>> >>> Sure. Optimization is for later. >>> (just to be sure, the limitation is on the 'column family', not on the >>> 'column') >> >> Yes, I am aware, thanks for reminding. >> >>> Just tell us if yo

Re: hbase-mailbox alpha release

2011-08-17 Thread Eric Charles
On 17/08/11 18:39, Ioan Eugen Stan wrote: 2011/8/17 Eric Charles: Sure. Optimization is for later. (just to be sure, the limitation is on the 'column family', not on the 'column') Yes, I am aware, thanks for reminding. Just tell us if you need help to progress on the integration tests. Maybe

Re: hbase-mailbox alpha release

2011-08-17 Thread Ioan Eugen Stan
2011/8/17 Eric Charles : > Sure. Optimization is for later. > (just to be sure, the limitation is on the 'column family', not on the > 'column') Yes, I am aware, thanks for reminding. > Just tell us if you need help to progress on the integration tests. > Maybe you could have more unit tests that

Re: hbase-mailbox alpha release

2011-08-17 Thread Eric Charles
Sure. Optimization is for later. (just to be sure, the limitation is on the 'column family', not on the 'column') Just tell us if you need help to progress on the integration tests. Maybe you could have more unit tests that will help you assess the functionality is correctly implemented: e.g.

Re: hbase-mailbox alpha release

2011-08-17 Thread Ioan Eugen Stan
2011/8/17 Eric Charles : > Hi Ioan, > > Didn't jump to my eyes reading the code, but schema showed clearly showed > it. > > HBase is known to outperform for with many column families per table > (http://hbase.apache.org/book/number.of.cfs.html), and you are using 5 > column families in the Messages

Re: hbase-mailbox alpha release

2011-08-17 Thread Eric Charles
Hi Ioan, Didn't jump to my eyes reading the code, but schema showed clearly showed it. HBase is known to outperform for with many column families per table (http://hbase.apache.org/book/number.of.cfs.html), and you are using 5 column families in the Messages table. I remind from HBase mail

Re: hbase-mailbox alpha release

2011-08-13 Thread Ioan Eugen Stan
2011/8/13 Eric Charles : > Hi Ioan, Hi Eric, nice to hear from you again. > > I've hg pull/update, and compile/test works fine here. Good ;) > > I didn't review the code, but integration tests will tell you if you are > missing something. How is it going? This is also a reply for Norman's mail:

Re: hbase-mailbox alpha release

2011-08-13 Thread Norman Maurer
Hi Ioan, any success with the integration tests yet ? bye norman Am Samstag, 13. August 2011 schrieb Eric Charles : > Hi Ioan, > > I've hg pull/update, and compile/test works fine here. Good ;) > > I didn't review the code, but integration tests will tell you if you are missing something. How is

Re: hbase-mailbox alpha release

2011-08-13 Thread Eric Charles
Hi Ioan, I've hg pull/update, and compile/test works fine here. Good ;) I didn't review the code, but integration tests will tell you if you are missing something. How is it going? You could also add in the tests: HBaseMailboxManagerTest extends AbstractMailboxManagerTest HBaseSubscriptionMan

Re: hbase-mailbox alpha release

2011-08-10 Thread Ioan Eugen Stan
Hello, Thanks to Norman's help, I have started on the integration tests. I also started on a Schema diagram as a MindMap. It is a work in progress but you can see a preview here [1]. Bye, [1] http://mailbox-hdfs.apache-extras.org.codespot.com/files/James%20HBase%20Mailbox%20Schema.png -- Ioan

Re: hbase-mailbox alpha release

2011-08-09 Thread Ioan Eugen Stan
> > Unfortunately, save is not done yet. It seems to use a lot of memory. > If I eliminate save() by returning SimpleMessageMetadate from the > start, the test is green so the problem lies here. Scratch that. I ran the tests again from the console, without NetBeans so my machine performed much bet

Re: hbase-mailbox alpha release

2011-08-09 Thread Ioan Eugen Stan
2011/8/9 Robert Burrell Donkin : > On Tue, Aug 9, 2011 at 3:17 PM, Ioan Eugen Stan wrote: >> 2011/8/9 Robert Burrell Donkin : >>> On Mon, Aug 8, 2011 at 9:57 PM, Ioan Eugen Stan >>> wrote: I fixed the exception in the test (forgot to update it when I made the change). I also fixed some

Re: hbase-mailbox alpha release

2011-08-09 Thread Robert Burrell Donkin
On Tue, Aug 9, 2011 at 3:17 PM, Ioan Eugen Stan wrote: > 2011/8/9 Robert Burrell Donkin : >> On Mon, Aug 8, 2011 at 9:57 PM, Ioan Eugen Stan >> wrote: >>> I fixed the exception in the test (forgot to update it when I made the >>> change). I also fixed some other issues. >>> Now all tests should

Re: hbase-mailbox alpha release

2011-08-09 Thread Ioan Eugen Stan
2011/8/9 Robert Burrell Donkin : > On Mon, Aug 8, 2011 at 9:57 PM, Ioan Eugen Stan wrote: >> I fixed the exception in the test (forgot to update it when I made the >> change). I also fixed some other issues. >> Now all tests should pass, except for HBaseMailboxManager which runs >> indefinitely. I

Re: hbase-mailbox alpha release

2011-08-09 Thread Robert Burrell Donkin
On Mon, Aug 8, 2011 at 9:57 PM, Ioan Eugen Stan wrote: > I fixed the exception in the test (forgot to update it when I made the > change). I also fixed some other issues. > Now all tests should pass, except for HBaseMailboxManager which runs > indefinitely. I tried to profile the test case in NetB

Re: hbase-mailbox alpha release

2011-08-08 Thread Norman Maurer
Hi Ioan, I'm currently very busy so first of sorry for the delay... I will have a look later today and also provide you with some help to get the code running in the mailbox integration test. I just need to fix a few things there first which broke the tests for now. As I added supported for C

Re: hbase-mailbox alpha release

2011-08-08 Thread Ioan Eugen Stan
I fixed the exception in the test (forgot to update it when I made the change). I also fixed some other issues. Now all tests should pass, except for HBaseMailboxManager which runs indefinitely. I tried to profile the test case in NetBeans but failed. I have an idea of what's going on, but not bein

Re: hbase-mailbox alpha release

2011-08-08 Thread Robert Burrell Donkin
On Mon, Aug 8, 2011 at 3:58 PM, Ioan Eugen Stan wrote: >> Have you tried running the protocol test suite? > > Nope, can you give me some tips on how to do it properly? Not sure (hopefully someone will jump in :-) With a little bit of luck, if we can build the mailbox and drop it into James then

Re: hbase-mailbox alpha release

2011-08-08 Thread Robert Burrell Donkin
On Mon, Aug 8, 2011 at 2:26 PM, Ioan Eugen Stan wrote: > Hello, > > It's your favorite GSoC student that wishes to make the following announce: > > hbase-mailbox implementation is feature complete and has entered Alpha > release (tag 0.3-alpha in the mercurial repository at [1]). > [1] > http:

Re: hbase-mailbox alpha release

2011-08-08 Thread Ioan Eugen Stan
> Have you tried running the protocol test suite? Nope, can you give me some tips on how to do it properly? Thanks, -- Ioan Eugen Stan http://ieugen.blogspot.com/ - To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.o

Re: hbase-mailbox alpha release

2011-08-08 Thread Robert Burrell Donkin
On Mon, Aug 8, 2011 at 2:26 PM, Ioan Eugen Stan wrote: > Hello, > > It's your favorite GSoC student that wishes to make the following announce: > > hbase-mailbox implementation is feature complete and has entered Alpha > release (tag 0.3-alpha in the mercurial repository at [1]). > I also wish to