2011/8/16 Norman Maurer <[email protected]>: > 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
Thanks Norman, I will apply the patch right away. -- Ioan Eugen Stan http://ieugen.blogspot.com/ --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
