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/apache/james/mailbox/hbase/mail/HBaseMessageMapper.java Tue Aug 16 09:58:02 2011 +0200
@@ -401,7 +401,9 @@
             scanner = messages.getScanner(scan);
             Result result = scanner.next();
             if (result == null) {
- throw new MailboxException("Search of first unseen message failed in mailbox " + mailbox);
+
+ // Return null as there are no unseen messages in the mailbox
+               return null;
             }
             return Bytes.toLong(result.getRow(), 16, 8);
         } catch (IOException e) {

I will do more review later..

Bye,
Norman



Am 16.08.2011 00:18, schrieb Ioan Eugen Stan:
Hello,

The integration tests are done. I have created a second repository for
them at [1]. You can get a working copy with the following command:
hg clone 
https://[email protected]/a/apache-extras.org/p/hbase-mailbox-integration-test/
.

I have runt the tests and some pass and other don't. Short version:
Tests run: 291, Failures: 0, Errors: 216, Skipped: 0
Mostly due to Cast exceptions and the like. I will see what I can do
to solve them. Help is appreciated.

[1] http://code.google.com/a/apache-extras.org/p/hbase-mailbox-integration-test/

Regards,



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to