from:"Eric Charles"

Eric Charles created JAMES-1478:
---

 Summary: Integrate spring related classes and configuration in on 
module
 Key: JAMES-1478
 URL: https://issues.apache.org/jira/browse/JAMES-1478
 Project: James Server
  Issue Type: Improvement
Reporter: Eric Charles
Assignee: Eric Charles


In mailbox, we have mailbox-spring.
For server, the spring stuff is under app (conf file), container-spring and 
lifecycle-spring.

I will migrate all under spring module.
The split of spring conf file can still be achieved and the spring conf file 
will import them via classpath.

In the future, I will also create the guice module responsible for guice 
injection.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-08 Thread Eric Charles

Of course, for external (non James) dependencies, we should only use 
RELEASE versions, except if a dev works in a sandbox or in a non 
enrolled module.


Thx, Eric

On 08/02/2013 10:02, Eric Charles wrote:

Hi Ioan,

I don't think it is a good strategy to forbid using SNAPSHOT because we
would stop integration development between subproject.

If I add a new feature in mailbox, it is to be able to use it in server.
And I want to commit my devs so that everybody can see them and and use
them.

Thx,

Eric

On 08/02/2013 08:46, Ioan Eugen Stan wrote:

Hello Eric,

Great, I'm looking forward for that commit. Not related to that, I
think we should adopt a policy NOT to use SNAPSHOT versions of our
artifacts in Server.
We should use releases and if we need to use a new version of
protocols or mailbox than we should make a new release of those
artifacts and then use them.

What do you think?

Cheers,



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1479) Create a Guice module

Eric Charles created JAMES-1479:
---

 Summary: Create a Guice module
 Key: JAMES-1479
 URL: https://issues.apache.org/jira/browse/JAMES-1479
 Project: James Server
  Issue Type: Improvement
  Components: Deployment Modules
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles


The guice module will allow to run the complete server with pure guice 
injection.
It will also allow to embed any James component in any 3rd party Guice 
application.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1478) Integrate spring related classes and configuration in on module


 [ 
https://issues.apache.org/jira/browse/JAMES-1478?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1478.
---

   Resolution: Fixed
Fix Version/s: 3.0-beta5

Done

 Integrate spring related classes and configuration in on module
 ---

 Key: JAMES-1478
 URL: https://issues.apache.org/jira/browse/JAMES-1478
 Project: James Server
  Issue Type: Improvement
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5


 In mailbox, we have mailbox-spring.
 For server, the spring stuff is under app (conf file), container-spring and 
 lifecycle-spring.
 I will migrate all under spring module.
 The split of spring conf file can still be achieved and the spring conf file 
 will import them via classpath.
 In the future, I will also create the guice module responsible for guice 
 injection.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Updated] (JAMES-1477) Replace @Resource with @Inject where it can


 [ 
https://issues.apache.org/jira/browse/JAMES-1477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles updated JAMES-1477:


Issue Type: Sub-task  (was: Improvement)
Parent: JAMES-1479

 Replace @Resource with @Inject where it can
 ---

 Key: JAMES-1477
 URL: https://issues.apache.org/jira/browse/JAMES-1477
 Project: James Server
  Issue Type: Sub-task
Reporter: Eric Charles
Assignee: Eric Charles

 This will allow also allow to work with different dependency injection 
 frameworks (spring, guice...).
 See also JAMES-1237.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1476) ArrayIndexOutOfBoundsException when searching in ImapFolder


[ 
https://issues.apache.org/jira/browse/JAMES-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13574385#comment-13574385
 ] 

Eric Charles commented on JAMES-1476:
-

Hi,
Is it possible to post/attach a way to replicate that issue.
We would love to solve it.

Thx, Eric

 ArrayIndexOutOfBoundsException when searching in ImapFolder 
 

 Key: JAMES-1476
 URL: https://issues.apache.org/jira/browse/JAMES-1476
 Project: James Server
  Issue Type: Bug
  Components: IMAPServer
Affects Versions: 3.0-beta4
Reporter: Heinz-Dieter Conradi
Priority: Critical

 Hi,
 Occasionally, some of my test fail with an ArrayIndexOutOfBoundsException:
 java.lang.ArrayIndexOutOfBoundsException: message number (6) out of bounds (5)
 at com.sun.mail.imap.MessageCache.getMessage(MessageCache.java:132)
 at com.sun.mail.imap.MessageCache.getMessageBySeqnum(MessageCache.java:162)
 at com.sun.mail.imap.IMAPFolder.getMessageBySeqNumber(IMAPFolder.java:2809)
 at com.sun.mail.imap.IMAPFolder.search(IMAPFolder.java:1662)
 The test is rather simple: I create a Message and do 
 'Transport.send(message)' and then I try to verify if the mail can be found 
 on the server with 'folder.search(searchTerm);'.
 The folder is opened with 'folder.open(Folder.READ_ONLY);'.
 I have asked on the javax.mail mailing list 
 (https://forums.oracle.com/forums/thread.jspa?threadID=2489711tstart=0) and 
 have been advised to open a bug here:
 --
 It's definitely a bug in the server. You can see it here:
 You open the INBOX and it tells you there's 6 messages in the INBOX:
 A3 EXAMINE INBOX
 * FLAGS (\Answered \Deleted \Draft \Flagged \Seen)
 * 6 EXISTS
 Then you do a search and it matches message number 7!
 A5 SEARCH SUBJECT subject with random nr: 1359967923405 9 ALL
 * SEARCH 7
 There was also another search for the same message right before that, and it 
 returned nothing.
 Presumably the message just showed up in the INBOX between those two 
 searches, which is
 fine, but the IMAP protocol requires the server to notify the client of the 
 new message before
 it can return that message as the result of a search.
 
 The complete stacktrace and debug output can be found in that thread, or I 
 can attach it to this bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (MAILBOX-77) Guice Mailbox Module


[ 
https://issues.apache.org/jira/browse/MAILBOX-77?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13574386#comment-13574386
 ] 

Eric Charles commented on MAILBOX-77:
-

See also JAMES-1479
(can not make subtask or link, MAILBOX and JAMES are different JIRA projects...)

 Guice Mailbox Module
 

 Key: MAILBOX-77
 URL: https://issues.apache.org/jira/browse/MAILBOX-77
 Project: James Mailbox
  Issue Type: New Feature
Reporter: Eric Charles
Assignee: Norman Maurer



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1476) ArrayIndexOutOfBoundsException when searching in ImapFolder


[ 
https://issues.apache.org/jira/browse/JAMES-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13574528#comment-13574528
 ] 

Eric Charles commented on JAMES-1476:
-

Btw, I run with maildir mailbox, which mailbox do you use? (if not maildir, you 
could try with maildir and see if you have the issue)

 ArrayIndexOutOfBoundsException when searching in ImapFolder 
 

 Key: JAMES-1476
 URL: https://issues.apache.org/jira/browse/JAMES-1476
 Project: James Server
  Issue Type: Bug
  Components: IMAPServer
Affects Versions: 3.0-beta4
Reporter: Heinz-Dieter Conradi
Priority: Critical
 Attachments: cdm-messaging-config-3.2-SNAPSHOT.jar, JamesTest.java


 Hi,
 Occasionally, some of my test fail with an ArrayIndexOutOfBoundsException:
 java.lang.ArrayIndexOutOfBoundsException: message number (6) out of bounds (5)
 at com.sun.mail.imap.MessageCache.getMessage(MessageCache.java:132)
 at com.sun.mail.imap.MessageCache.getMessageBySeqnum(MessageCache.java:162)
 at com.sun.mail.imap.IMAPFolder.getMessageBySeqNumber(IMAPFolder.java:2809)
 at com.sun.mail.imap.IMAPFolder.search(IMAPFolder.java:1662)
 The test is rather simple: I create a Message and do 
 'Transport.send(message)' and then I try to verify if the mail can be found 
 on the server with 'folder.search(searchTerm);'.
 The folder is opened with 'folder.open(Folder.READ_ONLY);'.
 I have asked on the javax.mail mailing list 
 (https://forums.oracle.com/forums/thread.jspa?threadID=2489711tstart=0) and 
 have been advised to open a bug here:
 --
 It's definitely a bug in the server. You can see it here:
 You open the INBOX and it tells you there's 6 messages in the INBOX:
 A3 EXAMINE INBOX
 * FLAGS (\Answered \Deleted \Draft \Flagged \Seen)
 * 6 EXISTS
 Then you do a search and it matches message number 7!
 A5 SEARCH SUBJECT subject with random nr: 1359967923405 9 ALL
 * SEARCH 7
 There was also another search for the same message right before that, and it 
 returned nothing.
 Presumably the message just showed up in the INBOX between those two 
 searches, which is
 fine, but the IMAP protocol requires the server to notify the client of the 
 new message before
 it can return that message as the result of a search.
 
 The complete stacktrace and debug output can be found in that thread, or I 
 can attach it to this bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1476) ArrayIndexOutOfBoundsException when searching in ImapFolder


[ 
https://issues.apache.org/jira/browse/JAMES-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13574527#comment-13574527
 ] 

Eric Charles commented on JAMES-1476:
-

Hi,
I have run your test class with the current James trunk default config (with my 
username/password) for +/- 5 minutes without exception (see end of the log here 
after).

As always, with intermittent failures, it is not obvious to reproduce.
Any particular conditions that would make this happen: parallel run of the 
test, machine load,... ?

Thx, Eric

DEBUG IMAP: AUTH: PLAIN
A3 EXAMINE INBOX
* FLAGS (\Answered \Deleted \Draft \Flagged \Seen NonJunk)
* 796 EXISTS
* 796 RECENT
* OK [UIDVALIDITY 321340841] UIDs valid
* OK [UNSEEN 796] Message 796 is first unseen
* OK [PERMANENTFLAGS (\Answered \Deleted \Draft \Flagged \Seen NonJunk \*)] 
Limited
* OK [HIGHESTMODSEQ 829] Highest
* OK [UIDNEXT 830] Predicted next UID
A3 OK [READ-ONLY] EXAMINE completed.
DEBUG IMAP: IMAPProtocol noop
A1478 NOOP
* 796 EXISTS
* 313 RECENT
A1478 OK NOOP completed.
A1479 SEARCH SUBJECT subject with random nr: 1360335549198 92 ALL
* SEARCH 796
A1479 OK SEARCH completed.
A1480 FETCH 796 (BODYSTRUCTURE)
* 796 FETCH (BODYSTRUCTURE (TEXT TXT (charset UTF-8) NIL NIL 7BIT 19 
1 NIL NIL NIL NIL))
A1480 OK FETCH completed.
A1481 FETCH 796 (BODY[TEXT]0.19)
* 796 FETCH (FLAGS (\Recent \Seen) BODY[TEXT]0 {19}
this is a message
)
A1481 OK FETCH completed.
[2013.02.08 GMT 14:59:32.877] (aos.protocol.imap4.ImapTest.java:118):testSend - 
msg found
DEBUG: getProvider() returning 
javax.mail.Provider[TRANSPORT,smtp,com.sun.mail.smtp.SMTPTransport,Sun 
Microsystems, Inc]
DEBUG SMTP: useEhlo true, useAuth true
DEBUG SMTP: useEhlo true, useAuth true
DEBUG SMTP: trying to connect to host localhost.net, port 2525, isSSL false
220 eric JAMES SMTP Server Server (JAMES SMTP Server ) ready
DEBUG SMTP: connected to host localhost.net, port: 2525

EHLO localhost
250-eric Hello localhost [127.0.0.1])
250-PIPELINING
250-ENHANCEDSTATUSCODES
250 8BITMIME
DEBUG SMTP: Found extension PIPELINING, arg 
DEBUG SMTP: Found extension ENHANCEDSTATUSCODES, arg 
DEBUG SMTP: Found extension 8BITMIME, arg 
DEBUG SMTP: use8bit false
MAIL FROM:tests...@icw.cm.test
250 2.1.0 Sender tests...@icw.cm.test OK
RCPT TO:e...@localhost.net
250 2.1.5 Recipient e...@localhost.net OK
DEBUG SMTP: Verified Addresses
DEBUG SMTP:   e...@localhost.net
DATA
354 Ok Send data ending with CRLF.CRLF
Date: Fri, 8 Feb 2013 14:47:50 + (GMT)
From: tests...@icw.cm.test
To: e...@localhost.net
Message-ID: 822082860.483.1360335572929.JavaMail.eric@eric
Subject: subject with random nr: 1360335572928 14
MIME-Version: 1.0
Content-Type: text/txt; charset=UTF-8
Content-Transfer-Encoding: 7bit

this is a message
.
250 2.6.0 Message received
QUIT
221 2.0.0 eric Service closing transmission channel
DEBUG: no connections in the pool, creating a new one
* OK JAMES IMAP4rev1 Server  Server eric is ready.
A0 CAPABILITY
* CAPABILITY SASL-IR IDLE LITERAL+ AUTH=PLAIN CONDSTORE ESEARCH CHILDREN 
UIDPLUS UNSELECT WITHIN ENABLE I18NLEVEL=1 SEARCHRES IMAP4rev1 NAMESPACE QRESYNC
A0 OK CAPABILITY completed.
DEBUG IMAP: AUTH: PLAIN
DEBUG IMAP: AUTHENTICATE PLAIN command trace suppressed
DEBUG IMAP: AUTHENTICATE PLAIN command result: A1 OK AUTHENTICATE completed.
A2 CAPABILITY
* CAPABILITY SASL-IR IDLE LITERAL+ AUTH=PLAIN CONDSTORE ESEARCH CHILDREN 
UIDPLUS UNSELECT WITHIN ENABLE I18NLEVEL=1 SEARCHRES IMAP4rev1 NAMESPACE QRESYNC
A2 OK CAPABILITY completed.
DEBUG IMAP: AUTH: PLAIN
A3 EXAMINE INBOX
* FLAGS (\Answered \Deleted \Draft \Flagged \Seen NonJunk)
* 797 EXISTS
* 797 RECENT
* OK [UIDVALIDITY 321340841] UIDs valid
* OK [UNSEEN 797] Message 797 is first unseen
* OK [PERMANENTFLAGS (\Answered \Deleted \Draft \Flagged \Seen NonJunk \*)] 
Limited
* OK [HIGHESTMODSEQ 830] Highest
* OK [UIDNEXT 831] Predicted next UID
A3 OK [READ-ONLY] EXAMINE completed.
DEBUG IMAP: IMAPProtocol noop
A1482 NOOP
* 797 EXISTS
* 313 RECENT
A1482 OK NOOP completed.
A1483 SEARCH SUBJECT subject with random nr: 1360335572928 14 ALL


 ArrayIndexOutOfBoundsException when searching in ImapFolder 
 

 Key: JAMES-1476
 URL: https://issues.apache.org/jira/browse/JAMES-1476
 Project: James Server
  Issue Type: Bug
  Components: IMAPServer
Affects Versions: 3.0-beta4
Reporter: Heinz-Dieter Conradi
Priority: Critical
 Attachments: cdm-messaging-config-3.2-SNAPSHOT.jar, JamesTest.java


 Hi,
 Occasionally, some of my test fail with an ArrayIndexOutOfBoundsException:
 java.lang.ArrayIndexOutOfBoundsException: message number (6) out of bounds (5)
 at com.sun.mail.imap.MessageCache.getMessage(MessageCache.java:132)
 at com.sun.mail.imap.MessageCache.getMessageBySeqnum(MessageCache.java:162

[jira] [Commented] (JAMES-1476) ArrayIndexOutOfBoundsException when searching in ImapFolder


[ 
https://issues.apache.org/jira/browse/JAMES-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13574550#comment-13574550
 ] 

Eric Charles commented on JAMES-1476:
-

Although much code is common to all mailbox implementations, each impl has its 
own responsibility to store, retrieve... based on UID.
In your case, based on what you say (load, redirect to one mailbox), I would 
suspect more a concurrency issue in the IMAP code base...
If we can reproduce, we should be able to bring a fix...
Thx, Eric

 ArrayIndexOutOfBoundsException when searching in ImapFolder 
 

 Key: JAMES-1476
 URL: https://issues.apache.org/jira/browse/JAMES-1476
 Project: James Server
  Issue Type: Bug
  Components: IMAPServer
Affects Versions: 3.0-beta4
Reporter: Heinz-Dieter Conradi
Priority: Critical
 Attachments: cdm-messaging-config-3.2-SNAPSHOT.jar, JamesTest.java


 Hi,
 Occasionally, some of my test fail with an ArrayIndexOutOfBoundsException:
 java.lang.ArrayIndexOutOfBoundsException: message number (6) out of bounds (5)
 at com.sun.mail.imap.MessageCache.getMessage(MessageCache.java:132)
 at com.sun.mail.imap.MessageCache.getMessageBySeqnum(MessageCache.java:162)
 at com.sun.mail.imap.IMAPFolder.getMessageBySeqNumber(IMAPFolder.java:2809)
 at com.sun.mail.imap.IMAPFolder.search(IMAPFolder.java:1662)
 The test is rather simple: I create a Message and do 
 'Transport.send(message)' and then I try to verify if the mail can be found 
 on the server with 'folder.search(searchTerm);'.
 The folder is opened with 'folder.open(Folder.READ_ONLY);'.
 I have asked on the javax.mail mailing list 
 (https://forums.oracle.com/forums/thread.jspa?threadID=2489711tstart=0) and 
 have been advised to open a bug here:
 --
 It's definitely a bug in the server. You can see it here:
 You open the INBOX and it tells you there's 6 messages in the INBOX:
 A3 EXAMINE INBOX
 * FLAGS (\Answered \Deleted \Draft \Flagged \Seen)
 * 6 EXISTS
 Then you do a search and it matches message number 7!
 A5 SEARCH SUBJECT subject with random nr: 1359967923405 9 ALL
 * SEARCH 7
 There was also another search for the same message right before that, and it 
 returned nothing.
 Presumably the message just showed up in the INBOX between those two 
 searches, which is
 fine, but the IMAP protocol requires the server to notify the client of the 
 new message before
 it can return that message as the result of a search.
 
 The complete stacktrace and debug output can be found in that thread, or I 
 can attach it to this bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1015) Configuration files of components should be moved from the spring-deployment module to the concrete implementation


[ 
https://issues.apache.org/jira/browse/JAMES-1015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13573322#comment-13573322
 ] 

Eric Charles commented on JAMES-1015:
-

Hi Ioan,

If you don't change them, then that's fine.
We have some doc to override : http://james.apache.org/server/3/config.html

Thx, Eric

 Configuration files of components should be moved from the spring-deployment 
 module to the concrete implementation
 --

 Key: JAMES-1015
 URL: https://issues.apache.org/jira/browse/JAMES-1015
 Project: James Server
  Issue Type: Sub-task
Reporter: Norman Maurer
Assignee: Ioan Eugen Stan

 At the moment all configuration files are in the spring-deployment module. 
 This does not make sense, we should move the configuration file to the 
 component it belongs to. So for example smtpserver.xml should get moved to 
 the smtpserver component / module. 
 This will make it easier to create a osgi deployment later too

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles


Hi Ioan,

Thx for this version 2.

What if we need to migrate to commons-configuration 1.9 to use a new 
functionality of that particular version whereas 
org.apache.servicemix.bundles.commons-configuration is only available 
for 1.8.* [1]?


Will it break karaf integration?

Thx, Eric


[1] 
http://repo1.maven.org/maven2/org/apache/servicemix/bundles/org.apache.servicemix.bundles.commons-configuration/


On 07/02/2013 09:04, Ioan Eugen Stan wrote:

Hello Eric,

Thanks for taking time for review.

On Wed, Feb 6, 2013 at 2:45 PM, Eric Charles e...@apache.org wrote:

It's possible to exclude the commons-configuration in the karaf module and
add there the org.apache.servicemix.bundles.commons-configuration
dependency.


I did something like that. I've built the feature to use the bundle
and depended on commons-configuration in the normal jars.
I've pushed the changes to [1] so now it should be ok to merge.


I guess one day commons-configuration will be 'bundled'.


Let's hope this will happen in version 2.

[1] https://github.com/ieugen/james/compare/karaf



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (IMAP-370) Consider supporting the upcoming MOVE extension

[
https://issues.apache.org/jira/browse/IMAP-370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13573357#comment-13573357
]

Eric Charles commented on IMAP-370:
---

Hi Andrzej,
Seems less easy than expected to use composition.
We still need to create the chain and for this each processor is built with it
predecessor [1].
I don't see directly how to tackle this, but with some free time, it should be
possible.
You will also need to run the integration test [2] to be sure everything is
fine, and also deploy the server and test with a real client (reading, moving
mails...)
That's the only way to as much as possible sure nothing is broken.

Thx, Eroc

[1]
https://svn.apache.org/repos/asf/james/protocols/trunk/imap/src/main/java/org/apache/james/imap/processor/DefaultProcessorChain.java
[2] https://svn.apache.org/repos/asf/james/mailbox-integration-tester/trunk

Consider supporting the upcoming MOVE extension
---

Key: IMAP-370
URL: https://issues.apache.org/jira/browse/IMAP-370
Project: James Imap
Issue Type: Improvement
Components: Protocol
Reporter: Andrzej Rusin
Attachments: IMAP-370-2.patch, IMAP-370-capability-v1.patch,
IMAP-370-v1.patch

As we see at
http://datatracker.ietf.org/doc/draft-ietf-imapmove-command/ballot/ IETF is
about to finally accept the IMAP MOVE extension.
This extension enables great performance optimization possibilities for the
more advanced Mailstore backends, eg the ones that use a underlying
relational or non-relational database:
One of the most common usage scenarios in IMAP is moving messages across
folders, and with the MOVE extention it can be finally accomplished without a
COPY/DELETE cycle on certain Mailbox backends.
Therefore, in my belief, it would be very nice to have that extension
implemented in James.
Some of the bundled Mailstore implementations already are based on SQL/noSQL
datasources, and can directly benefit from it.
Some of the independent Mailstore implementations (including mine) can
benefit from it too.
Some of IMAP clients (including Thunderbird as of 3.something) already
support that extension or the X-MOVE or X-AOL-MOVE one.
So James has a great opportunity to be the leader of standards adoption on
the server side.
What do you think? Please comment.

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1015) Configuration files of components should be moved from the spring-deployment module to the concrete implementation

[
https://issues.apache.org/jira/browse/JAMES-1015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13573385#comment-13573385
]

Eric Charles commented on JAMES-1015:
-

Hi Ioan,

the dns-context.xml was an early age attemp to do what you are achieving right
now.
So yes, splitting ((re)moving and importing) the spring-server.xml makes sense.

I just wanted to insist on the fact that the non-spring configuration files are
already at their places.
The job on the spring configuration files still needs to be done.
Feel free to take that way.

Thx, Eric

Configuration files of components should be moved from the spring-deployment
module to the concrete implementation
--

Key: JAMES-1015
URL: https://issues.apache.org/jira/browse/JAMES-1015
Project: James Server
Issue Type: Sub-task
Reporter: Norman Maurer
Assignee: Ioan Eugen Stan

At the moment all configuration files are in the spring-deployment module.
This does not make sense, we should move the configuration file to the
component it belongs to. So for example smtpserver.xml should get moved to
the smtpserver component / module.
This will make it easier to create a osgi deployment later too

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles


Thx Ioan.
Feel free to welcome karak in james trunk.
Eric

On 07/02/2013 10:06, Ioan Eugen Stan wrote:

Hello Eric,

We are actually running commons-configuration 1.9. I've created a
bundle for it via ServiceMix but I made a typo in the artifact name
[1], [2].
It's quite an easy thing to do to repackage a jar as a bundle ~ 5 min.
Look at [3].

Figures out we need to use the service mix artifact only in the
feature. The rest of the code can stay with the normal version.
I'm still considering replacing the way we do configuration in future
versions of James and I'm sure we can reach a solution that will work.

Plus: I've added a small readme:
https://github.com/ieugen/james/blob/karaf/karaf/README.md

[1] 
http://search.maven.org/#artifactdetails%7Corg.apache.servicemix.bundles%7Corg.apache.servicemmix.bundles.commons-configuration%7C1.9_1%7Cbundle
[2] https://issues.apache.org/jira/browse/SMX4-1340
[3] git://git.apache.org/servicemix4-bundles.git


On Thu, Feb 7, 2013 at 11:44 AM, Eric Charles e...@apache.org wrote:

Hi Ioan,

Thx for this version 2.

What if we need to migrate to commons-configuration 1.9 to use a new
functionality of that particular version whereas
org.apache.servicemix.bundles.commons-configuration is only available for
1.8.* [1]?

Will it break karaf integration?

Thx, Eric


[1]
http://repo1.maven.org/maven2/org/apache/servicemix/bundles/org.apache.servicemix.bundles.commons-configuration/


On 07/02/2013 09:04, Ioan Eugen Stan wrote:


Hello Eric,

Thanks for taking time for review.

On Wed, Feb 6, 2013 at 2:45 PM, Eric Charles e...@apache.org wrote:


It's possible to exclude the commons-configuration in the karaf module
and
add there the org.apache.servicemix.bundles.commons-configuration
dependency.



I did something like that. I've built the feature to use the bundle
and depended on commons-configuration in the normal jars.
I've pushed the changes to [1] so now it should be ok to merge.


I guess one day commons-configuration will be 'bundled'.



Let's hope this will happen in version 2.

[1] https://github.com/ieugen/james/compare/karaf



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org







-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: svn commit: r1443402 - in /james/server/trunk: ./ container-spring/ data/data-file/ data/data-hbase/ data/data-jcr/ data/data-jdbc/ data/data-jpa/ data/data-ldap/ data/data-library/ data/data-libr

2013-02-07 Thread Eric Charles


Hi Ioan,
Why do you replace commons-configuration in container-spring.
This is not what we said.
Please revert.

Thx, Eric


On 07/02/2013 10:58, ieu...@apache.org wrote:

Author: ieugen
Date: Thu Feb  7 10:58:31 2013
New Revision: 1443402

URL: http://svn.apache.org/viewvc?rev=1443402view=rev
Log:
OSGi love

* commons-configuration feature is installable in Karaf
* replaced commmons-configuration dependency with repacked version from 
org.apache.servicemix.bundles.commons-configuration

Modified:
 james/server/trunk/container-spring/pom.xml
 james/server/trunk/data/data-file/pom.xml
 james/server/trunk/data/data-hbase/pom.xml
 james/server/trunk/data/data-jcr/pom.xml
 james/server/trunk/data/data-jdbc/pom.xml
 james/server/trunk/data/data-jpa/pom.xml
 james/server/trunk/data/data-ldap/pom.xml
 james/server/trunk/data/data-library/pom.xml
 
james/server/trunk/data/data-library/src/main/java/org/apache/james/domainlist/lib/AbstractDomainList.java
 james/server/trunk/dns-service/dnsservice-dnsjava/pom.xml
 james/server/trunk/fetchmail/pom.xml
 james/server/trunk/karaf/features/pom.xml
 james/server/trunk/karaf/features/src/main/resources/features.xml
 james/server/trunk/lifecycle-api/pom.xml
 james/server/trunk/lifecycle-spring/pom.xml
 james/server/trunk/mailetcontainer-camel/pom.xml
 james/server/trunk/pom.xml
 james/server/trunk/protocols/protocols-imap4/pom.xml
 james/server/trunk/protocols/protocols-library/pom.xml
 james/server/trunk/protocols/protocols-lmtp/pom.xml
 james/server/trunk/protocols/protocols-pop3/pom.xml
 james/server/trunk/protocols/protocols-smtp/pom.xml

Modified: james/server/trunk/container-spring/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/container-spring/pom.xml?rev=1443402r1=1443401r2=1443402view=diff
==
--- james/server/trunk/container-spring/pom.xml (original)
+++ james/server/trunk/container-spring/pom.xml Thu Feb  7 10:58:31 2013
@@ -109,8 +109,8 @@
  scopeprovided/scope
  /dependency
  dependency
-groupIdcommons-configuration/groupId
-artifactIdcommons-configuration/artifactId
+groupIdorg.apache.servicemix.bundles/groupId
+
artifactIdorg.apache.servicemix.bundles.commons-configuration/artifactId
  /dependency
  dependency
  groupIdcommons-collections/groupId

Modified: james/server/trunk/data/data-file/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/data/data-file/pom.xml?rev=1443402r1=1443401r2=1443402view=diff
==
--- james/server/trunk/data/data-file/pom.xml (original)
+++ james/server/trunk/data/data-file/pom.xml Thu Feb  7 10:58:31 2013
@@ -60,8 +60,8 @@
  artifactIdapache-mailet-api/artifactId
  /dependency
  dependency
-groupIdcommons-configuration/groupId
-artifactIdcommons-configuration/artifactId
+groupIdorg.apache.servicemix.bundles/groupId
+
artifactIdorg.apache.servicemix.bundles.commons-configuration/artifactId
  /dependency
  dependency
  groupIdorg.slf4j/groupId

Modified: james/server/trunk/data/data-hbase/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/data/data-hbase/pom.xml?rev=1443402r1=1443401r2=1443402view=diff
==
--- james/server/trunk/data/data-hbase/pom.xml (original)
+++ james/server/trunk/data/data-hbase/pom.xml Thu Feb  7 10:58:31 2013
@@ -49,8 +49,8 @@
  artifactIdjames-server-data-library/artifactId
  /dependency
  dependency
-groupIdcommons-configuration/groupId
-artifactIdcommons-configuration/artifactId
+groupIdorg.apache.servicemix.bundles/groupId
+
artifactIdorg.apache.servicemix.bundles.commons-configuration/artifactId
  /dependency
  dependency
  groupIdorg.slf4j/groupId

Modified: james/server/trunk/data/data-jcr/pom.xml
URL: 
http://svn.apache.org/viewvc/james/server/trunk/data/data-jcr/pom.xml?rev=1443402r1=1443401r2=1443402view=diff
==
--- james/server/trunk/data/data-jcr/pom.xml (original)
+++ james/server/trunk/data/data-jcr/pom.xml Thu Feb  7 10:58:31 2013
@@ -44,8 +44,8 @@
  artifactIdjames-server-data-library/artifactId
  /dependency
  dependency
-groupIdcommons-configuration/groupId
-artifactIdcommons-configuration/artifactId
+groupIdorg.apache.servicemix.bundles/groupId
+
artifactIdorg.apache.servicemix.bundles.commons-configuration/artifactId
  /dependency
  dependency

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles


Hi Ioan,

See my reply on previous mail.

org.apache.servicemix.bundles.commons-configuration should only be 
declared in karaf modules.


Thx, Eric


On 07/02/2013 11:06, Ioan Eugen Stan wrote:

Hello,

Thanks Eric. It should be committed in a few minutes. Could you give
me some clues as to how components depend on one-another?

DNSService has the least dependencies, which should I try to add next? Queues?

Cheers,



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles


Hi Ioan,

It's even worth than I tought.
There are now some org.apache.servicemix.bundles.commons-configuration 
all over the place in all modules.


I for now revert, exclude commons-configuration and redeclare 
org.apache.servicemix.bundles.commons-configuration in the karaf modules.


Thx, Eric


On 07/02/2013 11:08, Eric Charles wrote:

Hi Ioan,

See my reply on previous mail.

org.apache.servicemix.bundles.commons-configuration should only be
declared in karaf modules.

Thx, Eric


On 07/02/2013 11:06, Ioan Eugen Stan wrote:

Hello,

Thanks Eric. It should be committed in a few minutes. Could you give
me some clues as to how components depend on one-another?

DNSService has the least dependencies, which should I try to add next?
Queues?

Cheers,



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles




On 07/02/2013 11:48, Eric Charles wrote:

Hi Ioan,

It's even worth than I tought.


read worse, not worth



There are now some org.apache.servicemix.bundles.commons-configuration
all over the place in all modules.

I for now revert, exclude commons-configuration and redeclare
org.apache.servicemix.bundles.commons-configuration in the karaf modules.

Thx, Eric


On 07/02/2013 11:08, Eric Charles wrote:

Hi Ioan,

See my reply on previous mail.

org.apache.servicemix.bundles.commons-configuration should only be
declared in karaf modules.

Thx, Eric


On 07/02/2013 11:06, Ioan Eugen Stan wrote:

Hello,

Thanks Eric. It should be committed in a few minutes. Could you give
me some clues as to how components depend on one-another?

DNSService has the least dependencies, which should I try to add next?
Queues?

Cheers,



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1477) Replace @Resource with @Inject where it can

Eric Charles created JAMES-1477:
---

 Summary: Replace @Resource with @Inject where it can
 Key: JAMES-1477
 URL: https://issues.apache.org/jira/browse/JAMES-1477
 Project: James Server
  Issue Type: Improvement
Reporter: Eric Charles
Assignee: Eric Charles


This will allow also allow to work with different dependency injection 
frameworks (spring, guice...).

See also JAMES-1237.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-07 Thread Eric Charles


Hi Ioan,

Thx for this.

I have a long time pending branch related to JAMES-1237 where last 
comment was replacing @Resource with @Inject to prepare guice injection.


I will now commit a first version and will continue to fix the last bits 
tomorrow.


I have create JAMES-1477 to track down this.

Thx, Eric


On 07/02/2013 14:44, Ioan Eugen Stan wrote:

Hello Eric,

I've committed the changes and switched to commons-configuration as it
should have been. Next, I'm moving to fix app to import spring
configs.

Cheers,



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1015) Configuration files of components should be moved from the spring-deployment module to the concrete implementation

2013-02-06 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/JAMES-1015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13572361#comment-13572361
 ] 

Eric Charles commented on JAMES-1015:
-

This is already done Ioan for the non-OSGI files.
Please leave them where they are.


 Configuration files of components should be moved from the spring-deployment 
 module to the concrete implementation
 --

 Key: JAMES-1015
 URL: https://issues.apache.org/jira/browse/JAMES-1015
 Project: James Server
  Issue Type: Sub-task
Reporter: Norman Maurer
Assignee: Ioan Eugen Stan

 At the moment all configuration files are in the spring-deployment module. 
 This does not make sense, we should move the configuration file to the 
 component it belongs to. So for example smtpserver.xml should get moved to 
 the smtpserver component / module. 
 This will make it easier to create a osgi deployment later too

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-06 Thread Eric Charles


Hi Ioan,

I see commons-configuration is removed in favor of 
org.apache.servicemix.bundles.commons-configuration.


Is there no way to define that only for karaf distribution.
I don't see why the non-osgi part should depend on a servicemix 
packaging. What if we want to update commons-configuration and that 
servicemix is not updated.


I am -1 for now regarding replacing commons-configuration with 
org.apache.servicemix.bundles.commons-configuration in the modules.


Thx, Eric


On 06/02/2013 12:00, Ioan Eugen Stan wrote:

Hello Devs,

I've made some progress regarding James-Karaf integration and I'm
ready to commit the work I've done in trunk. You can find the
change-set for review here [1].
To view a diff between trunk/karaf, check out [2] .

It adds the following under karaf folder:

* features - creates a Karaf features XML artifact that allows us to
easily install James components as a standalone unit in Karaf
* distribution - will create a custom james-karaf
* integration - integration tests for features

Right now, we can install only a few features - commons-configuration,
mime4j and dnsservice. The good part is that all things are set into
place and things should move faster since the groundwork is done.

Cheers,

[1] https://github.com/ieugen/james/tree/karaf
[2] https://github.com/ieugen/james/compare/karaf



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: james karaf commit in trunk

2013-02-06 Thread Eric Charles

It's possible to exclude the commons-configuration in the karaf module 
and add there the org.apache.servicemix.bundles.commons-configuration 
dependency.


I guess one day commons-configuration will be 'bundled'.

Thx, Eric

On 06/02/2013 12:34, Eric Charles wrote:

Hi Ioan,

I see commons-configuration is removed in favor of
org.apache.servicemix.bundles.commons-configuration.

Is there no way to define that only for karaf distribution.
I don't see why the non-osgi part should depend on a servicemix
packaging. What if we want to update commons-configuration and that
servicemix is not updated.

I am -1 for now regarding replacing commons-configuration with
org.apache.servicemix.bundles.commons-configuration in the modules.

Thx, Eric


On 06/02/2013 12:00, Ioan Eugen Stan wrote:

Hello Devs,

I've made some progress regarding James-Karaf integration and I'm
ready to commit the work I've done in trunk. You can find the
change-set for review here [1].
To view a diff between trunk/karaf, check out [2] .

It adds the following under karaf folder:

* features - creates a Karaf features XML artifact that allows us to
easily install James components as a standalone unit in Karaf
* distribution - will create a custom james-karaf
* integration - integration tests for features

Right now, we can install only a few features - commons-configuration,
mime4j and dnsservice. The good part is that all things are set into
place and things should move faster since the groundwork is done.

Cheers,

[1] https://github.com/ieugen/james/tree/karaf
[2] https://github.com/ieugen/james/compare/karaf



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (IMAP-370) Consider supporting the upcoming MOVE extension

2013-02-06 Thread Eric Charles (JIRA)

[
https://issues.apache.org/jira/browse/IMAP-370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13572539#comment-13572539
]

Eric Charles commented on IMAP-370:
---

Hi Andrzej,
I was thinking to create a new MoveProcessor(new CopyProcessor) rather than
having MoveProcessor extends CopyProcessor.
So yes, I was meaning Composition design pattern.
With that, you respect isAcceptable() and you can reuse the CopyProcessor
methods from your MoveProcessor.
Thx, Eric

Consider supporting the upcoming MOVE extension
---

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (IMAP-370) Consider supporting the upcoming MOVE extension

2013-02-06 Thread Eric Charles (JIRA)

[
https://issues.apache.org/jira/browse/IMAP-370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13572761#comment-13572761
]

Eric Charles commented on IMAP-370:
---

Yeah Andrzej, this is what I meant.
I will apply it tomorrow.
Thx, Eric

Consider supporting the upcoming MOVE extension
---

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1474) @PersistenceContext annotations should refer to unit name 'James'


[ 
https://issues.apache.org/jira/browse/JAMES-1474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13571290#comment-13571290
 ] 

Eric Charles commented on JAMES-1474:
-

Sound logical Marco.
Patch welcome as always :)
Eric

 @PersistenceContext annotations should refer to unit name 'James'
 -

 Key: JAMES-1474
 URL: https://issues.apache.org/jira/browse/JAMES-1474
 Project: James Server
  Issue Type: Bug
  Components: Spring Container
Affects Versions: Trunk
Reporter: Marco Zapletal
Priority: Minor
   Original Estimate: 0.5h
  Remaining Estimate: 0.5h

 The apache-james-mailbox-jpa project uses @PersistenceContext annotations for 
 injecting the entity manager. As 'James' is used as the persistence unit name 
 throughout the project, the @PersisteceContext annotations should refer to 
 the corresponding unit name.
 This would allow to integrate James with custom code that uses a different 
 entity manager factory.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (MAILBOX-191) more effective implementation of StoreMessageResultIterator


[ 
https://issues.apache.org/jira/browse/MAILBOX-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13571463#comment-13571463
 ] 

Eric Charles commented on MAILBOX-191:
--

Hi Peter,

I have applied the patch and it seems to work.
However, I have issues with trunk disconnecting the connection.
I am looking at that issue before changing anything on the 
StoreMessageResultIterator.

Thx again, Eric

 more effective implementation of StoreMessageResultIterator
 ---

 Key: MAILBOX-191
 URL: https://issues.apache.org/jira/browse/MAILBOX-191
 Project: James Mailbox
  Issue Type: Improvement
  Components: store
Affects Versions: 0.4, 0.5
Reporter: Peter Kvokacka
Priority: Minor
  Labels: patch
 Fix For: 0.5

 Attachments: StoreMessageResultIterator.java, 
 StoreMessageResultIterator.patch


 I found problem with current implementation of  StoreMessageResultIterator. I 
 discribed it here.
 http://www.mail-archive.com/server-user@james.apache.org/msg14242.html
 Briefly:
 From Thunderbird I got request
452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])
 which is mapped to calls on MessageMapper.findInMailbox()
findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full
 Those marked calls of method findInMailbox are completely unnecessary and 
 should be avoided.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (MAILBOX-193) Bump to jackrabbit 2.5.2 to benefit from lucene 3.6.0 and reactive lucene module

Eric Charles created MAILBOX-193:


 Summary: Bump to jackrabbit 2.5.2 to benefit from lucene 3.6.0 and 
reactive lucene module
 Key: MAILBOX-193
 URL: https://issues.apache.org/jira/browse/MAILBOX-193
 Project: James Mailbox
  Issue Type: Task
Affects Versions: 0.4
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 0.4




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (MAILBOX-193) Bump to jackrabbit 2.5.2 to benefit from lucene 3.6.0 and reactive lucene module


 [ 
https://issues.apache.org/jira/browse/MAILBOX-193?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed MAILBOX-193.


Resolution: Fixed

Done.

 Bump to jackrabbit 2.5.2 to benefit from lucene 3.6.0 and reactive lucene 
 module
 

 Key: MAILBOX-193
 URL: https://issues.apache.org/jira/browse/MAILBOX-193
 Project: James Mailbox
  Issue Type: Task
Affects Versions: 0.4
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 0.4




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1475) Add unit test class and related resources configuration to james-server-app

Eric Charles created JAMES-1475:
---

 Summary: Add unit test class and related resources configuration 
to james-server-app
 Key: JAMES-1475
 URL: https://issues.apache.org/jira/browse/JAMES-1475
 Project: James Server
  Issue Type: Task
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5


This will allow to ensure server starts and also to have resource to easily run 
server from eclipse with non-root friendly ports.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1460) Simplify pom and update some dependencies


[ 
https://issues.apache.org/jira/browse/JAMES-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13572215#comment-13572215
 ] 

Eric Charles commented on JAMES-1460:
-

We now build with-assembly by default.
This is how it should be to be sure we deliver valid tar and to ease devs life.

 Simplify pom and update some dependencies
 -

 Key: JAMES-1460
 URL: https://issues.apache.org/jira/browse/JAMES-1460
 Project: James Server
  Issue Type: Sub-task
  Components: Build System
Affects Versions: 3.0-beta4
Reporter: Ioan Eugen Stan
Assignee: Ioan Eugen Stan



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (IMAP-370) Consider supporting the upcoming MOVE extension

[
https://issues.apache.org/jira/browse/IMAP-370?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13572220#comment-13572220
]

Eric Charles commented on IMAP-370:
---

Hi Andrzej,
I finally had to remove the MoveProcessor from the chain as it was breaking the
connection when a copy command was issued by the client.
This is due to the AbstractChainedProcessor.isAcceptable method which relies on
Class.isInstance to see if the processor is valid for the command.
In this particular case, as Move extends Copy, the condition is fullfilled and
the move is assigned to the copy.
The ideal would be to use composition instead of extension.
I am happy to discuss any valid proposal (change the isAcceptable logic...)
although I would prefer the composition approach keeping the isAcceptable as it
is now.

Thx, Eric

Consider supporting the upcoming MOVE extension
---

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (MAILBOX-191) more effective implementation of StoreMessageResultIterator


 [ 
https://issues.apache.org/jira/browse/MAILBOX-191?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed MAILBOX-191.


Resolution: Fixed
  Assignee: Eric Charles

Hi Peter,
In the meantime, I have resolved the disconnection issues (related to 
IMAP-370), and I run with your patch in production.

I have committed it in trunk.

If I further see any issue, I will reopen this JIRA.

Thx a lot,
Eric

 more effective implementation of StoreMessageResultIterator
 ---

 Key: MAILBOX-191
 URL: https://issues.apache.org/jira/browse/MAILBOX-191
 Project: James Mailbox
  Issue Type: Improvement
  Components: store
Affects Versions: 0.4, 0.5
Reporter: Peter Kvokacka
Assignee: Eric Charles
Priority: Minor
  Labels: patch
 Fix For: 0.5

 Attachments: StoreMessageResultIterator.java, 
 StoreMessageResultIterator.patch


 I found problem with current implementation of  StoreMessageResultIterator. I 
 discribed it here.
 http://www.mail-archive.com/server-user@james.apache.org/msg14242.html
 Briefly:
 From Thunderbird I got request
452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])
 which is mapped to calls on MessageMapper.findInMailbox()
findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full
 Those marked calls of method findInMailbox are completely unnecessary and 
 should be avoided.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1474) @PersistenceContext annotations should refer to unit name 'James'


 [ 
https://issues.apache.org/jira/browse/JAMES-1474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1474.
---

   Resolution: Fixed
Fix Version/s: 3.0-beta5
 Assignee: Eric Charles

Committed to trunk. Thx Marco.

 @PersistenceContext annotations should refer to unit name 'James'
 -

 Key: JAMES-1474
 URL: https://issues.apache.org/jira/browse/JAMES-1474
 Project: James Server
  Issue Type: Bug
  Components: Spring Container
Affects Versions: Trunk
Reporter: Marco Zapletal
Assignee: Eric Charles
Priority: Minor
 Fix For: 3.0-beta5

 Attachments: JAMES-1474.patch

   Original Estimate: 0.5h
  Remaining Estimate: 0.5h

 The apache-james-mailbox-jpa project uses @PersistenceContext annotations for 
 injecting the entity manager. As 'James' is used as the persistence unit name 
 throughout the project, the @PersisteceContext annotations should refer to 
 the corresponding unit name.
 This would allow to integrate James with custom code that uses a different 
 entity manager factory.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (MAILBOX-191) more effective implementation of StoreMessageResultIterator

2013-02-02 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/MAILBOX-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13569731#comment-13569731
 ] 

Eric Charles commented on MAILBOX-191:
--

Hi Peter,

Sorry for this late reply...

For the patch, this is what I did:

eric:trunk svn st
eric:trunk $ cd ..
eric:james.svn eric$ patch -p0  StoreMessageResultIterator.java 
(Stripping trailing CRs from patch.)
patching file 
trunk/store/src/main/java/org/apache/james/mailbox/store/StoreMessageResultIterator.java
Hunk #1 FAILED at 38.
Hunk #2 FAILED at 50.
Hunk #3 FAILED at 112.
Hunk #4 FAILED at 146.
4 out of 4 hunks FAILED -- saving rejects to file 
trunk/store/src/main/java/org/apache/james/mailbox/store/StoreMessageResultIterator.java.rej
eric:james.svn eric$ more 
trunk/store/src/main/java/org/apache/james/mailbox/store/StoreMessageResultIterator.java.rej
***
*** 38,43 
  import org.apache.james.mailbox.store.mail.model.Mailbox;
  import org.apache.james.mailbox.store.mail.model.Message;
  
  public class StoreMessageResultIteratorId implements MessageResultIterator {
  
  private IteratorMessageId next = null;
--- 38,44 
  import org.apache.james.mailbox.store.mail.model.Mailbox;
  import org.apache.james.mailbox.store.mail.model.Message;
  



For the integration tests, you can checkout the code from 
https://svn.apache.org/repos/asf/james/mailbox-integration-tester/trunk/

Run after 'mvn test', it will run the tests for all the mailbox implementations

Hope this helps  Thx in advance,

Eric

 more effective implementation of StoreMessageResultIterator
 ---

 Key: MAILBOX-191
 URL: https://issues.apache.org/jira/browse/MAILBOX-191
 Project: James Mailbox
  Issue Type: Improvement
  Components: store
Affects Versions: 0.4, 0.5
Reporter: Peter Kvokacka
Priority: Minor
  Labels: patch
 Fix For: 0.5

 Attachments: StoreMessageResultIterator.java


 I found problem with current implementation of  StoreMessageResultIterator. I 
 discribed it here.
 http://www.mail-archive.com/server-user@james.apache.org/msg14242.html
 Briefly:
 From Thunderbird I got request
452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])
 which is mapped to calls on MessageMapper.findInMailbox()
findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full
 Those marked calls of method findInMailbox are completely unnecessary and 
 should be avoided.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1472) ReferenceStore not valid ... / Mail spooled but not processed

2013-01-30 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/JAMES-1472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13566472#comment-13566472
 ] 

Eric Charles commented on JAMES-1472:
-

Hi Markus,

In beta3, yes, james-server-context.xml is the file.

But if I remember well, FileMailQueue was introduced between beta3 and beta4, 
with fixes in trunk (so even not released).

Thx, Eric

 ReferenceStore not valid ... / Mail spooled but not processed
 -

 Key: JAMES-1472
 URL: https://issues.apache.org/jira/browse/JAMES-1472
 Project: James Server
  Issue Type: Bug
  Components: Remote Delivery, SpoolManager  Processors
Affects Versions: 3.0-beta3
 Environment: Red Hat Enterprise Linux Server release 5.7 (Tikanga), 
Reporter: Markus Moldaschl
 Attachments: dump1.log, dump2.log, dump3.log


 see 
 http://mail-archives.apache.org/mod_mbox/james-server-user/201301.mbox/browser
  for the description of the problem

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1472) ReferenceStore not valid ... / Mail spooled but not processed

2013-01-27 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/JAMES-1472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13563771#comment-13563771
 ] 

Eric Charles commented on JAMES-1472:
-

Hi Markus,

I have analyzed the 3 given thread dumps with IBMC JCA tool.
They all show remote delivery waiting thread, which is the normal state when 
the threads are waiting on mails from ActiveMQ queues.
All the other thread seem normal: no deadlock, no huge number of spooling 
threads that could indicate something wrong...

If you want to give a try to the FileQueue to see if the issue if related to 
ActiveMQ or not (could be something else in your system/env), you can update 
the META-INF/org/apache/james/spring-server.xml :

comment out bean id=mailqueuefactory 
class=org.apache.james.queue.file.FileMailQueueFactory /

comment in bean id=mailqueuefactory 
class=org.apache.james.queue.activemq.ActiveMQMailQueueFactory 
depends-on=broker /


Thx, Eric

 ReferenceStore not valid ... / Mail spooled but not processed
 -

 Key: JAMES-1472
 URL: https://issues.apache.org/jira/browse/JAMES-1472
 Project: James Server
  Issue Type: Bug
  Components: Remote Delivery, SpoolManager  Processors
Affects Versions: 3.0-beta3
 Environment: Red Hat Enterprise Linux Server release 5.7 (Tikanga), 
Reporter: Markus Moldaschl
 Attachments: dump1.log, dump2.log, dump3.log


 see 
 http://mail-archives.apache.org/mod_mbox/james-server-user/201301.mbox/browser
  for the description of the problem

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (MAILBOX-191) more effective implementation of StoreMessageResultIterator

2013-01-27 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/MAILBOX-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13563772#comment-13563772
 ] 

Eric Charles commented on MAILBOX-191:
--

I Peter,

I tried to apply the patch on mailbox trunk with 'patch -p0...', but got some 
rejections.
Did you build it on the trunk version?

I am happy to give it more try based on your inputs.

The logic may be good, but I really want to run the mailbox integration tests. 
Did you try to run those tests with your patch?
https://svn.apache.org/repos/asf/james/mailbox-integration-tester/trunk/

Also, always great to have unit tests that cover the class 
StoreMessageResultIterator

Thx, Eric

 more effective implementation of StoreMessageResultIterator
 ---

 Key: MAILBOX-191
 URL: https://issues.apache.org/jira/browse/MAILBOX-191
 Project: James Mailbox
  Issue Type: Improvement
  Components: store
Affects Versions: 0.4, 0.5
Reporter: Peter Kvokacka
Priority: Minor
  Labels: patch
 Fix For: 0.5

 Attachments: StoreMessageResultIterator.java


 I found problem with current implementation of  StoreMessageResultIterator. I 
 discribed it here.
 http://www.mail-archive.com/server-user@james.apache.org/msg14242.html
 Briefly:
 From Thunderbird I got request
452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])
 which is mapped to calls on MessageMapper.findInMailbox()
findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full
 Those marked calls of method findInMailbox are completely unnecessary and 
 should be avoided.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (MAILBOX-192) Upgrade mailbox to james-project 1.8.2-SNAPSHOT

2013-01-26 Thread Eric Charles (JIRA)

Eric Charles created MAILBOX-192:


 Summary: Upgrade mailbox to james-project 1.8.2-SNAPSHOT
 Key: MAILBOX-192
 URL: https://issues.apache.org/jira/browse/MAILBOX-192
 Project: James Mailbox
  Issue Type: Bug
Affects Versions: 0.4
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 0.4


Upgrade to latest parent and remove definition for test-jar as they are now in 
parent.
Remove also clean plugin as it breaks the build via cli and user also does not 
want to have its file out of /target removed by maven...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (MAILBOX-192) Upgrade mailbox to james-project 1.8.2-SNAPSHOT

2013-01-26 Thread Eric Charles (JIRA)


 [ 
https://issues.apache.org/jira/browse/MAILBOX-192?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed MAILBOX-192.


Resolution: Fixed

Done.

 Upgrade mailbox to james-project 1.8.2-SNAPSHOT
 ---

 Key: MAILBOX-192
 URL: https://issues.apache.org/jira/browse/MAILBOX-192
 Project: James Mailbox
  Issue Type: Bug
Affects Versions: 0.4
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 0.4


 Upgrade to latest parent and remove definition for test-jar as they are now 
 in parent.
 Remove also clean plugin as it breaks the build via cli and user also does 
 not want to have its file out of /target removed by maven...

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1436) APPEND IMAP command can result in JAMES IMAP waiting indefinitely for data

2013-01-20 Thread Eric Charles (JIRA)

[
https://issues.apache.org/jira/browse/JAMES-1436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13558292#comment-13558292
]

Eric Charles commented on JAMES-1436:
-

Hi Jerry,
Thx a lot for your all the explanations.
Based on these, I have committed the rebased patch to trunk.
http://svn.apache.org/viewvc?rev=1435870view=rev

Thx also to Sarmant for the initial patch.

Let's leave this JIRA open for a while to see if we don't have any side effects
and also to be sure everything is fixed for this.

Eric

APPEND IMAP command can result in JAMES IMAP waiting indefinitely for data
--

Key: JAMES-1436
URL: https://issues.apache.org/jira/browse/JAMES-1436
Project: James Server
Issue Type: Bug
Components: IMAPServer
Affects Versions: Trunk, 3.0-beta4, 3.0-beta5
Environment: Ubuntu 12.04 x86_64
Reporter: Samant Maharaj
Attachments: JAMES-1436.patch, JAMES-1436.patch.r1432540,
ThunderbirdAndIMAPserver.log

When processing an IMAP APPEND command, the Netty stack in JAMES IMAP can get
into a state where the ImapRequestFrameDecoder will wait for a number message
bytes that will never arrive.
This has the effect of causing the IMAP client to also block indefinitely
waiting for a response from the server.
Root Cause:
This is due to a race condition when the DelimiterBasedFrameDecoder is
removed from the Netty pipeline by the ImapRequestFrameDecoder.
If the DelimiterBasedFrameDecoder still contains less than one line of data
in its buffer, that data will never be flushed and forwarded down the
pipeline. The effect of this is that a small number of bytes, typically from
the early part of the message are omitted and the final byte count does not
match the value calculated from the APPEND command. This results in the
APPEND command never being completely decoded and hence no append actually
takes place nor does a response get sent to the client.
In order to reliably trigger this bug, JAMES was configured to accept a
remote debugging connection and a conditional breakpoint set at
org.jboss.netty.handler.codec.frame.FrameDecoder:439. The condition was set
to 'Thread.sleep(200l); false'. This results in introducing a 200ms delay on
each frame decoding loop without actually hitting the breakpoint. The effect
of this is to allow the threadpool running ImapRequestFrameDecoder time to
consume the individual frames and remove the DelimiterBasedFrameDecoder from
the pipeline.

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1471) Revert back to xml extension for configuration files

2013-01-20 Thread Eric Charles (JIRA)


 [ 
https://issues.apache.org/jira/browse/JAMES-1471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1471.
---

Resolution: Fixed

Done.

 Revert back to xml extension for configuration files
 

 Key: JAMES-1471
 URL: https://issues.apache.org/jira/browse/JAMES-1471
 Project: James Server
  Issue Type: Improvement
  Components: Deployment Modules
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5


 We will use '.xml' file extension for configuration file instead of '.conf'.
 This is in line with all the other java oss projects and allows to use other 
 format (yaml..) if one likes it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1471) Revert back to xml extension for configuration files

2013-01-19 Thread Eric Charles (JIRA)

Eric Charles created JAMES-1471:
---

 Summary: Revert back to xml extension for configuration files
 Key: JAMES-1471
 URL: https://issues.apache.org/jira/browse/JAMES-1471
 Project: James Server
  Issue Type: Improvement
  Components: Deployment Modules
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5


We will use '.xml' file extension for configuration file instead of '.conf'.
This is in line with all the other java oss projects and allows to use other 
format (yaml..) if one likes it.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Failover Bug with ActiveMQ

2013-01-18 Thread Eric Charles


Hi Philipp,

Thx again for you detailed inputs.
Yes, 1/ seems promising.

With a bit of effort, I'm sure it would even be possible to setup a unit 
test that launches 2 embedded amq, the first one being stopped after a 
delay (see [1] for the current unit test). You would also need to 
configure your broker configuration to read from both embedded amq 
(aBroker.addConnector(tcp://127.0.0.1);


The advantage of that kind of unit test is:

- You will get faster a candidate jar because developing is 
eclipse/unitest is much faster than deploying a new jar in james 
(star/stop) and stopping a real amq instance.

- You are more confident on your jars when you will try it in the real env.
- Other users will benefit from your unit test.

Whatever you do, Good luck :)

Thx, Eric

[1] 
https://svn.apache.org/repos/asf/james/server/trunk/queue/queue-activemq/src/test/java/org/apache/james/queue/activemq/ActiveMQMailQueueTest.java


On 17/01/2013 15:01, Philipp Krenn wrote:

Hi Eric,

thanks for the quick reply!

Just to be clear, depending on the configuration, we run into different 
exceptions:
1) If we use AMQ's PooledConnectionFactory we get a ClassCastException as soon 
as a message is being delivered due to the wrong exception. Failover is working 
fine, however.
2) If we use Spring's CachingConnectionFactory message delivery is working 
fine, but upon failover we run into an IllegalStateException which results in 
100% CPU utilization.

Our configuration looks like this:

##
!-- ActiveMQ: Broker, Connections, BlobTransferPolicy... --
!-- amq:broker useJmx=true persistent=false brokerName=james 
dataDirectory=filesystem=file://var/store/activemq/brokers
useShutdownHook=false schedulerSupport=false id=broker 
amq:destinationPolicy
amq:policyMap amq:policyEntries Support priority handling 
of messages
http://activemq.apache.org/how-can-i-support-priority-queues.html 
amq:policyEntry
queue= prioritizedMessages=true/ /amq:policyEntries 
/amq:policyMap
/amq:destinationPolicy amq:managementContext amq:managementContext 
createConnector=false/
/amq:managementContext amq:persistenceAdapter 
amq:amqPersistenceAdapter/
/amq:persistenceAdapter amq:plugins amq:statisticsBrokerPlugin/ 
/amq:plugins
amq:transportConnectors amq:transportConnector 
uri=tcp://localhost:6000
/ /amq:transportConnectors /amq:broker --
amq:connectionFactory id=amqConnectionFactory
brokerURL=${jms.broker.url}
amq:prefetchPolicy
!-- Disable prefetch so slow consuming can not block 
other threads --
!-- See JAMES-1253 --
amq:prefetchPolicy queuePrefetch=0
topicPrefetch=0 /
/amq:prefetchPolicy
property name=blobTransferPolicy ref=blobTransferPolicy /
/amq:connectionFactory
bean id=blobTransferPolicy

class=org.apache.james.queue.activemq.FileSystemBlobTransferPolicy
property name=defaultUploadUrl 
value=file://var/store/activemq/blob-transfer /
/bean
bean id=jmsConnectionFactory

class=org.springframework.jms.connection.CachingConnectionFactory
property name=targetConnectionFactory 
ref=amqConnectionFactory /
property name=sessionCacheSize value=10 /
property name=cacheConsumers value=false /
property name=cacheProducers value=false /
property name=reconnectOnException value=true /
/bean
!--
bean id=jmsConnectionFactory 
class=org.apache.activemq.pool.PooledConnectionFactory
destroy-method=stop
property name=connectionFactory ref=amqConnectionFactory
/property
/bean --
##

We'll look into 1) and will try to provide a patch.
We're not sure how / why 2) happens, but that's also not the intended behavior 
IMHO.

Cheers,
Philipp


Am 17.01.2013 um 14:32 schrieb Eric Charles e...@apache.org:


Hi Philipp,
Good to see you are progressing.
If I undertand well fail-over occurs, but your receive a ClassCastException 
upon receiving a message.

I would be great if you could share the details of your configuration (just 
replace hostname, username, password...) to make fail-over work.

Also, you could try to patch the code with javax.jms.ConnectionFactory, rebuild 
it and see if it works with the new jars.

Thx again and come back here if you need more details on anything,

Eric

On 17/01/2013 13:23, Philipp Krenn wrote:

Hi,

we have removed the definition for creating an ActiveMQ broker from the 
spring-server.xml and are using an external one. In order to do so, we have 
configured amq:connectionFactory.../ accordingly

[jira] [Commented] (MAILBOX-191) more effective implementation of StoreMessageResultIterator

2013-01-18 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/MAILBOX-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557300#comment-13557300
 ] 

Eric Charles commented on MAILBOX-191:
--

Hi Peter,
I will further look at you patch.
I feel we miss some unit tests.
Thx, Eric

 more effective implementation of StoreMessageResultIterator
 ---

 Key: MAILBOX-191
 URL: https://issues.apache.org/jira/browse/MAILBOX-191
 Project: James Mailbox
  Issue Type: Improvement
  Components: store
Affects Versions: 0.4, 0.5
Reporter: Peter Kvokacka
Priority: Minor
  Labels: patch
 Fix For: 0.5

 Attachments: StoreMessageResultIterator.java


 I found problem with current implementation of  StoreMessageResultIterator. I 
 discribed it here.
 http://www.mail-archive.com/server-user@james.apache.org/msg14242.html
 Briefly:
 From Thunderbird I got request
452 UID fetch 16,17:18,20:41,43:45,47,49 (UID RFC822.SIZE BODY.PEEK[])
 which is mapped to calls on MessageMapper.findInMailbox()
findInMailbox: TYPE: RANGE UID: 16:18, Full
*findInMailbox: TYPE: FROM UID: 19:-1, Full
findInMailbox: TYPE: RANGE UID: 20:41, Full
*findInMailbox: TYPE: FROM UID: 42:-1, Full
findInMailbox: TYPE: RANGE UID: 43:45, Full
*findInMailbox: TYPE: FROM UID: 46:-1, Full
findInMailbox: TYPE: ONE UID: 47:47, Full
*findInMailbox: TYPE: ONE UID: 48:48, Full
findInMailbox: TYPE: ONE UID: 49:49, Full
 Those marked calls of method findInMailbox are completely unnecessary and 
 should be avoided.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Failover Bug with ActiveMQ

2013-01-17 Thread Eric Charles


Hi Philipp,
Good to see you are progressing.
If I undertand well fail-over occurs, but your receive a 
ClassCastException upon receiving a message.


I would be great if you could share the details of your configuration 
(just replace hostname, username, password...) to make fail-over work.


Also, you could try to patch the code with javax.jms.ConnectionFactory, 
rebuild it and see if it works with the new jars.


Thx again and come back here if you need more details on anything,

Eric

On 17/01/2013 13:23, Philipp Krenn wrote:

Hi,

we have removed the definition for creating an ActiveMQ broker from the 
spring-server.xml and are using an external one. In order to do so, we have 
configured amq:connectionFactory.../ accordingly. This configuration is the 
same as we are using for several other camel-based applications (where we also use 
Spring's CachingConnectionFactory) accessing the same broker.

So we do not expect that this is a configuration error. What we also observed 
is that by replacing the CachingConnectionFactory by AMQ's 
PooledConnectionFactory, the failover works fine. However, an exception is 
thrown upon receiving a message, since somewhere in the code an 
org.springframework.jms.connection.CachingConnectionFactory instead of a 
javax.jms.ConnectionFactory is injected.

Best regards,
Philipp


Am 15.01.2013 um 20:57 schrieb Eric Charles e...@apache.org:


Hi,

Interestting, this is the first time I hear about a real James with amq in a 
failover configuration.

I guess you changed in spring-server.xml the block
amq:transportConnectors
   amq:transportConnector uri=tcp://localhost:0 /
/amq:transportConnectors

Maybe a first test scenario would be to be sure James doesn't use the queue, 
failingover amq, and begin to send again mail to james. The error may be due 
because the fail over occurs while a transaction is occurring?

Thx, Eric


On 15/01/2013 15:26, Philipp Krenn wrote:

Hi,


We are using James Server with an ActiveMQ master/slave configuration 
configured via failover: 
failover:(tcp://mq0.xxx:8080,tcp://mq1.xxx:8080)?jms.prefetchPolicy.all=0
When the master stops, we would expect a failover to the slave - this is 
working fine with other applications. However, James Server does not switch 
queue instance and keeps producing the following exception instead:

[ERROR] - Exception caught in RemoteDelivery.run() [Remote delivery thread (3)] 
[j.mailetcontext]
org.apache.james.queue.api.MailQueue$MailQueueException: Unable to dequeue next 
message (javax.jms.IllegalStateException: The Session is closed)
at 
org.apache.james.queue.jms.JMSMailQueue.deQueue(JMSMailQueue.java:170) 
~[james-server-queue-jms-3.0-beta5-20130109.104230-723.jar:3.0-beta5-SNAPSHOT]
at 
org.apache.james.transport.mailets.RemoteDelivery.run(RemoteDelivery.java:771) 
~[james-server-mailets-3.0-beta5-20130110.114225-802.jar:3.0-beta5-SNAPSHOT]
at java.lang.Thread.run(Thread.java:679) [na:1.6.0_24]
Caused by: javax.jms.IllegalStateException: The Session is closed
at 
org.apache.activemq.ActiveMQSession.checkClosed(ActiveMQSession.java:731) 
~[activemq-core-5.7.0.jar:5.7.0]
at 
org.apache.activemq.ActiveMQSession.createQueue(ActiveMQSession.java:1169) 
~[activemq-core-5.7.0.jar:5.7.0]
at sun.reflect.GeneratedMethodAccessor51.invoke(Unknown Source) ~[na:na]
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 ~[na:1.6.0_24]
at java.lang.reflect.Method.invoke(Method.java:616) ~[na:1.6.0_24]
at 
org.springframework.jms.connection.CachingConnectionFactory$CachedSessionInvocationHandler.invoke(CachingConnectionFactory.java:344)
 ~[spring-jms-3.1.0.RELEASE.jar:3.1.0.RELEASE]
at $Proxy87.createQueue(Unknown Source) ~[na:na]
at 
org.apache.james.queue.jms.JMSMailQueue.deQueue(JMSMailQueue.java:107) 
~[james-server-queue-jms-3.0-beta5-20130109.104230-723.jar:3.0-beta5-SNAPSHOT]
... 2 common frames omitted

The James Server keeps running and consumes 100% of the available CPU until it 
is manually restarted.

Thanks for looking into this issue.


Cheers,
Philipp


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: bounce mail messages

2013-01-15 Thread Eric Charles


Hi Rajender,

I read, but I really have not cycles for now to dig into your code.
Sorry for this... maybe someone else here can help?

Thx, Eric

On 15/01/2013 09:23, Rajender Vallapureddy wrote:

Hi Eric,

The problem I am seeing is if I use MimeMultipartReport, I am unable
to get any text in the body part.

I tried using MimeMultipart my mail is coming in as
disposition-notification and again If I setSubType to report, I am
unable to get any text in the body part.

Also, the messageType is either going as unknown or IMF. What
parameter/attribute I should look at to set messageType to dsn.

What i need is just to send a failed report and ghost the original mail.

I am using james 3 beta4 version.

MimeMultipart multipart = new 
MimeMultipart(mixed);
multipart.setSubType(report);

MimeBodyPart part = new 
MimeBodyPart();
part.setContent(messageString, 
message/rfc822);

part.setHeader(RFC2822Headers.CONTENT_TYPE,
delivery-status);
multipart.addBodyPart(part);


newMail.getMessage().setContent(multipart);

newMail.getMessage().setHeader(report-type,
delivery-status);


getMailetContext().sendMail(newMail);

Please appreciate your help.

Thank you,
Regards,
Rajender

On Mon, Jan 14, 2013 at 1:36 PM, Eric Charles e...@apache.org wrote:

Hi Rajender,

I tried on my local trunk and get the original message attached to the
bounce. It's true that my client (thunderbird) does not allow to open the
attachement as a real mail, but shows it as plain text.

Can you check the source of the bounce mail and see if this is not an issue
with the mail client you are using.

Thx, Eric


On 13/01/2013 15:37, Rajender Vallapureddy wrote:


Hi Eric,

I replicated the DNSBounce to send custom bounce with postmaster..

I am able to send / receive the postmaster message, but I am unable
see any body message in it.

I am using mysql and in the james_mail.mail_bytes field all I am
seeing is empty blob and when I try to open that mail in squirrel mail
its giving me the imap error.


My class is exact the DNSBounce.java except for few conditions when to
send the bounce mail.

MailImpl newMail = new MailImpl(mail);
newMail.setRemoteAddr(dns.getLocalHost().getHostName());
newMail.setRemoteHost(dns.getLocalHost().getHostAddress());
MailAddress reversePath = mail.getSender();
CollectionMailAddress newRecipients = new HashSetMailAddress();
newRecipients.add(reversePath);
newMail.setRecipients(newRecipients);
MimeMessage newMessage = new
MimeMessage(Session.getDefaultInstance(System.getProperties(), null));
MimeMultipartReport multipart = new MimeMultipartReport();
multipart.setReportType(delivery-status);
MimeBodyPart part1 = new MimeBodyPart();
part1.setText(messageString); // message from this part is missing in
the mail body.
multipart.addBodyPart(part1);
newMessage.setHeader(RFC2822Headers.CONTENT_TYPE,
multipart.getContentType());
newMessage.setContent(multipart);
newMail.setMessage(newMessage);
setRecipients(newMail, getRecipients(mail), mail);
setTo(newMail, getTo(mail), mail);
setSubjectPrefix(newMail, getSubjectPrefix(mail), mail);
newMail.getMessage().setHeader(RFC2822Headers.DATE,
rfc822DateFormat.format(new Date()));
setReplyTo(newMail, getReplyTo(mail), mail);
setReversePath(newMail, getReversePath(mail), mail);
setSender(newMail, getSender(mail), mail);
setIsReply(newMail, isReply(mail), mail);
newMail.getMessage().saveChanges();
getMailetContext().sendMail(newMail);


Please suggest me what is causing the body message part to be missing
from the mail send.

Appreciate your help.

Thank you,
Regards,
Rajender

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Failover Bug with ActiveMQ

2013-01-15 Thread Eric Charles


Hi,

Interestting, this is the first time I hear about a real James with amq 
in a failover configuration.


I guess you changed in spring-server.xml the block
amq:transportConnectors
   amq:transportConnector uri=tcp://localhost:0 /
/amq:transportConnectors

Maybe a first test scenario would be to be sure James doesn't use the 
queue, failingover amq, and begin to send again mail to james. The error 
may be due because the fail over occurs while a transaction is occurring?


Thx, Eric


On 15/01/2013 15:26, Philipp Krenn wrote:

Hi,


We are using James Server with an ActiveMQ master/slave configuration 
configured via failover: 
failover:(tcp://mq0.xxx:8080,tcp://mq1.xxx:8080)?jms.prefetchPolicy.all=0
When the master stops, we would expect a failover to the slave - this is 
working fine with other applications. However, James Server does not switch 
queue instance and keeps producing the following exception instead:

[ERROR] - Exception caught in RemoteDelivery.run() [Remote delivery thread (3)] 
[j.mailetcontext]
org.apache.james.queue.api.MailQueue$MailQueueException: Unable to dequeue next 
message (javax.jms.IllegalStateException: The Session is closed)
at 
org.apache.james.queue.jms.JMSMailQueue.deQueue(JMSMailQueue.java:170) 
~[james-server-queue-jms-3.0-beta5-20130109.104230-723.jar:3.0-beta5-SNAPSHOT]
at 
org.apache.james.transport.mailets.RemoteDelivery.run(RemoteDelivery.java:771) 
~[james-server-mailets-3.0-beta5-20130110.114225-802.jar:3.0-beta5-SNAPSHOT]
at java.lang.Thread.run(Thread.java:679) [na:1.6.0_24]
Caused by: javax.jms.IllegalStateException: The Session is closed
at 
org.apache.activemq.ActiveMQSession.checkClosed(ActiveMQSession.java:731) 
~[activemq-core-5.7.0.jar:5.7.0]
at 
org.apache.activemq.ActiveMQSession.createQueue(ActiveMQSession.java:1169) 
~[activemq-core-5.7.0.jar:5.7.0]
at sun.reflect.GeneratedMethodAccessor51.invoke(Unknown Source) ~[na:na]
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 ~[na:1.6.0_24]
at java.lang.reflect.Method.invoke(Method.java:616) ~[na:1.6.0_24]
at 
org.springframework.jms.connection.CachingConnectionFactory$CachedSessionInvocationHandler.invoke(CachingConnectionFactory.java:344)
 ~[spring-jms-3.1.0.RELEASE.jar:3.1.0.RELEASE]
at $Proxy87.createQueue(Unknown Source) ~[na:na]
at 
org.apache.james.queue.jms.JMSMailQueue.deQueue(JMSMailQueue.java:107) 
~[james-server-queue-jms-3.0-beta5-20130109.104230-723.jar:3.0-beta5-SNAPSHOT]
... 2 common frames omitted

The James Server keeps running and consumes 100% of the available CPU until it 
is manually restarted.

Thanks for looking into this issue.


Cheers,
Philipp



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: bounce mail messages

2013-01-14 Thread Eric Charles


Hi Rajender,

I tried on my local trunk and get the original message attached to the 
bounce. It's true that my client (thunderbird) does not allow to open 
the attachement as a real mail, but shows it as plain text.


Can you check the source of the bounce mail and see if this is not an 
issue with the mail client you are using.


Thx, Eric

On 13/01/2013 15:37, Rajender Vallapureddy wrote:

Hi Eric,

I replicated the DNSBounce to send custom bounce with postmaster..

I am able to send / receive the postmaster message, but I am unable
see any body message in it.

I am using mysql and in the james_mail.mail_bytes field all I am
seeing is empty blob and when I try to open that mail in squirrel mail
its giving me the imap error.


My class is exact the DNSBounce.java except for few conditions when to
send the bounce mail.

MailImpl newMail = new MailImpl(mail);
newMail.setRemoteAddr(dns.getLocalHost().getHostName());
newMail.setRemoteHost(dns.getLocalHost().getHostAddress());
MailAddress reversePath = mail.getSender();
CollectionMailAddress newRecipients = new HashSetMailAddress();
newRecipients.add(reversePath);
newMail.setRecipients(newRecipients);
MimeMessage newMessage = new
MimeMessage(Session.getDefaultInstance(System.getProperties(), null));
MimeMultipartReport multipart = new MimeMultipartReport();
multipart.setReportType(delivery-status);
MimeBodyPart part1 = new MimeBodyPart();
part1.setText(messageString); // message from this part is missing in
the mail body.
multipart.addBodyPart(part1);
newMessage.setHeader(RFC2822Headers.CONTENT_TYPE, multipart.getContentType());
newMessage.setContent(multipart);
newMail.setMessage(newMessage);
setRecipients(newMail, getRecipients(mail), mail);
setTo(newMail, getTo(mail), mail);
setSubjectPrefix(newMail, getSubjectPrefix(mail), mail);
newMail.getMessage().setHeader(RFC2822Headers.DATE,
rfc822DateFormat.format(new Date()));
setReplyTo(newMail, getReplyTo(mail), mail);
setReversePath(newMail, getReversePath(mail), mail);
setSender(newMail, getSender(mail), mail);
setIsReply(newMail, isReply(mail), mail);
newMail.getMessage().saveChanges();
getMailetContext().sendMail(newMail);


Please suggest me what is causing the body message part to be missing
from the mail send.

Appreciate your help.

Thank you,
Regards,
Rajender

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: factor-out commons-configuration

2013-01-13 Thread Eric Charles

Hi Ioan,

We use the HierarchicalConfiguration as its name implies, has some
notion of hierarchy in the configuration.

It also allows to move from XML to YAML for example (btw, I think we
should revert back from .conf to .xml to allow different
configuration formats).

Using simple properties file is possible with some fancy key naming. But
if we are only able to use those properties/preferences file, I think we
are downgrading the current powerfull solution we have we
commons-configuration.

About the OSGI option, I don't know really what this implies. I would
simply say the requirement for James that it is not needed to run in a
OSGI (Karaf...) container. OSGI Love is not mandatory for all our users.

Also, moving to something else must IMHO bring some added value such as
hot-reload, which is more a injection framework issue that a
configuration format/storage.

The best thing to do is to develop this in a branch so we don't impact
the curent code base as this will make all instable (build, working,
documentation...) for some time. Just committing such difficult parts in
trunk is not an option.

Thx, Eric

On 13/01/2013 06:35, Ioan Eugen Stan wrote:

Hello,

I've added Jean Baptiste and Robert Munteanu, who are not subscribed.
JB, Robert, feel free to not participate if you are not
interested/don't have time.

Eric, your help is needed.

I wish to refactor out commons-configuration in favor for a more OSGi
friendly option. I'm still researching a good solution and I'm also
open to suggestions. The main driver is to make it more OSGi friendly,
simplify things under the constraints defined by [a].
Commons-configuration brings a lot of commons-* dependencies which I
would like to see gone (less is more).

I have two options: java.util.prefs.Preferences [1]- [6] and OSGi
Configuration Admin [7]-[9].

Configuration admin has a simple interface but depend on one osgi
artifact, while Preferences are standard Java. Other than that they
seem to be equal with regard to features, also support what
commons-configuration has to offer. I believe support all the
restrictions defined by [a].

Thank you,

Important James Issues related to this:
[a] https://issues.apache.org/jira/browse/JAMES-495

[1]
http://docs.oracle.com/javase/1.4.2/docs/api/java/util/prefs/Preferences.html
[2]
http://docs.oracle.com/javase/1.5.0/docs/guide/preferences/index.html#prefs-other
[3] http://code.google.com/p/java-util-prefs-jdbc-backend/
[4]
http://devlearnings.wordpress.com/2010/11/12/use-java-util-prefs-preferences-instead-of-java-util-properties/
[5]
http://www.davidc.net/programming/java/java-preferences-using-file-backing-store
[6]
http://docs.oracle.com/javase/1.5.0/docs/api/java/util/prefs/Preferences.html#addNodeChangeListener(java.util.prefs.NodeChangeListener)

[7] http://www.osgi.org/javadoc/r4v42/org/osgi/service/cm/ManagedService.html
[8]
http://stackoverflow.com/questions/10225467/how-to-externalize-the-configuration-of-bundles
[9] http://felix.apache.org/site/apache-felix-config-admin.html

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1468) site document update of build instructions with trunk

2013-01-13 Thread Eric Charles (JIRA)


 [ 
https://issues.apache.org/jira/browse/JAMES-1468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1468.
---

Resolution: Fixed
  Assignee: Eric Charles

Committed and site deployed, thx Jerry

 site document update of build instructions with trunk
 -

 Key: JAMES-1468
 URL: https://issues.apache.org/jira/browse/JAMES-1468
 Project: James Server
  Issue Type: Improvement
  Components: Documentation
Affects Versions: 3.0-beta4
Reporter: Jerry Tian
Assignee: Eric Charles
  Labels: documentation
 Fix For: 3.0-beta5

 Attachments: dev-build.xml.diff


 make the build instructions more accurate with current trunk.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1470) Add apache snapshots repository

2013-01-12 Thread Eric Charles (JIRA)

Eric Charles created JAMES-1470:
---

 Summary: Add apache snapshots repository
 Key: JAMES-1470
 URL: https://issues.apache.org/jira/browse/JAMES-1470
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles


When building james from an empty maven repo, we miss the james-project 
snapshot.
As temp workaround, we need to add the apache.snapshor repository in the pom.
This can be removed on release (not obliged).
Eric

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Build failure on James trunk

2013-01-12 Thread Eric Charles


In that case, I ususally do mvn clean install and it works.
Add -DskipTests because we have for now test failure in the 
mailbox-integration-tests module.


Thx, Eric

On 11/01/2013 08:25, Jerry Tian wrote:

Hello,

I do think the build trunk should be put into Jenkins, but in a less frequent 
way, like one build a week or one build every two weeks? Maybe the tests can be 
disable too. If building without tests, the whole process should take less than 
5 minutes, at least on my Macbook(a 2012 model).

Unluckily, after issued an update today, the trunk fails again. ;-(

[INFO] 
[INFO] Building Apache JAMES Mime4j (DOM) 0.8-SNAPSHOT
[INFO] 
[INFO]
[INFO] --- javacc-maven-plugin:2.6:jjtree-javacc (generate-jjtree) @ 
apache-mime4j-dom ---
[INFO] Skipping - all parsers are up to date
[INFO]
[INFO] --- javacc-maven-plugin:2.6:javacc (generate-javacc) @ apache-mime4j-dom 
---
[INFO] Skipping - all parsers are up to date
[INFO]
[INFO] --- maven-remote-resources-plugin:1.2.1:process (default) @ 
apache-mime4j-dom ---
[INFO]
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
apache-mime4j-dom ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 3 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ 
apache-mime4j-dom ---
[INFO] Compiling 65 source files to 
/Users/jerry/Documents/james.workspace/apache-james-trunk/mime4j/dom/target/classes
[INFO] -
[ERROR] COMPILATION ERROR :
[INFO] -
[ERROR] 
/Users/jerry/Documents/james.workspace/apache-james-trunk/mime4j/dom/target/generated-sources/javacc/org/apache/james/mime4j/field/address/AddressListParser.java:[64,36]
 cannot access Token
class file for Token not found
 ((SimpleNode) n).firstToken = getToken(1);
[ERROR] 
/Users/jerry/Documents/james.workspace/apache-james-trunk/mime4j/dom/target/generated-sources/javacc/org/apache/james/mime4j/field/address/AddressListParser.java:[68,45]
 incompatible types
found   : org.apache.james.mime4j.field.address.Token
required: Token
[INFO] 2 errors
[INFO] -
[INFO] 
[INFO] Reactor Summary:
[INFO]
[INFO] Apache James Project .. SUCCESS [3.987s]
[INFO] Apache James :: Mailets parent and aggregator . SUCCESS [0.450s]
[INFO] Apache James :: Mailet API  SUCCESS [0.796s]
[INFO] Apache James :: Server  SUCCESS [0.372s]
[INFO] Apache James :: Server :: Data  :: API  SUCCESS [1.761s]
[INFO] Apache James :: Server :: Lifecycle API ... SUCCESS [0.335s]
[INFO] Apache James :: JSieve  SUCCESS [0.229s]
[INFO] Apache James :: jSieve :: Manager . SUCCESS [0.014s]
[INFO] Apache James :: jSieve :: Manager API . SUCCESS [0.146s]
[INFO] Apache James :: Server :: Filesystem API .. SUCCESS [0.957s]
[INFO] Apache James :: Server :: Lifecycle Spring  SUCCESS [0.904s]
[INFO] Apache James :: Server :: Mailetcontainer API . SUCCESS [0.826s]
[INFO] Apache James :: Server :: Common Utilities  SUCCESS [1.133s]
[INFO] Apache James :: Protocols . SUCCESS [0.214s]
[INFO] Apache James :: Protocols :: API .. SUCCESS [0.467s]
[INFO] Apache James :: Protocols :: Netty Implementation . SUCCESS [0.317s]
[INFO] Apache James :: Server :: Protocols Library ... SUCCESS [1.161s]
[INFO] Apache James :: Basic Mailet Toolkit .. SUCCESS [0.341s]
[INFO] Apache James :: Server :: Core  SUCCESS [0.520s]
[INFO] Apache James :: Mailbox ... SUCCESS [0.199s]
[INFO] Apache James :: Mailbox :: API  SUCCESS [1.126s]
[INFO] Apache James :: Mailbox :: Store Framework  SUCCESS [1.327s]
[INFO] Apache James :: Mailbox :: In Memory .. SUCCESS [0.537s]
[INFO] Apache James :: Mailbox :: Tools .. SUCCESS [0.825s]
[INFO] Apache James :: Server :: Container Spring  SUCCESS [1.182s]
[INFO] Apache James :: Server :: Client .. SUCCESS [0.851s]
[INFO] Apache James :: Server :: DNS Service :: API .. SUCCESS [0.754s]
[INFO] Apache James :: Server :: DNS Service :: Library .. SUCCESS [0.294s]
[INFO] Apache James :: Server :: Mail Queue :: API ... SUCCESS [0.554s]
[INFO] Apache James :: Standard Mailets .. SUCCESS [0.349s]
[INFO] Apache James :: Server :: Mailetcontainer Camel ... SUCCESS [0.782s]
[INFO] Apache James :: Mailbox :: Maildir

Re: James 3 beta 4 POM.xml has multiple locations of configuration files

2013-01-12 Thread Eric Charles


Hi Sai,
I guess WEB-INF/classes is better.

Feel free to submit a patch (also please ensure that the change does not 
break the normal tar.gz packaging).


Thx, Eric


On 11/01/2013 17:28, Saibabu Vallurupalli wrote:

Hi All,

I am making a WAR out of James 3.0 Beta 4 version and found the property
files are in two locations pom.xml file has something shown below making
this duplicate copy.

*
resource
 directorysrc/main/resources/directory
  !-- override the destination directory for this resource
--
 targetPathWEB-INF/conf//targetPath
 excludes
 excludelog4j.properties/exclude
 exclude**/lib/**/exclude
 /excludes
 /resource
 resource
 directorysrc/main/resources/directory
  !-- override the destination directory for this resource
--
 targetPathWEB-INF/classes//targetPath
 includes
 includelog4j.properties/include
 /includes
 /resource
***

When I am deploying the WAR into my application server, it is throwing
errors saying duplication of properties. Which is the preferred location of
properties so that we can avoid duplication.

Please advise.

Thank you,
Sai



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1470) Add apache snapshots repository

2013-01-12 Thread Eric Charles (JIRA)


[ 
https://issues.apache.org/jira/browse/JAMES-1470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13551877#comment-13551877
 ] 

Eric Charles commented on JAMES-1470:
-

Hi Iona,
Between releases, it's perfectly fine to depend on releases.
Yes, sure, we can release project
Project under server doesn't sound good because project is used by all other 
james components.
Thx, Eric

 Add apache snapshots repository
 ---

 Key: JAMES-1470
 URL: https://issues.apache.org/jira/browse/JAMES-1470
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles

 When building james from an empty maven repo, we miss the james-project 
 snapshot.
 As temp workaround, we need to add the apache.snapshor repository in the pom.
 This can be removed on release (not obliged).
 Eric

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Build failure on James trunk

2013-01-09 Thread Eric Charles


You're more than welcome :)
Feel free to submit the patch, I will happily commit it.

Thx, Eric


On 09/01/2013 07:46, Jerry Tian wrote:

cheers, finally catch it.

btw, if it helps to fix the document, please let me know. I found that they are in 
server/src/site/xdoc/dev-build.xml , so if needed, I can submit a patch of it.

Thanks.



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: svn commit: r1430296 - /james/project/trunk/pom.xml

2013-01-09 Thread Eric Charles


Hi Ioan,
In the meantime I have committed a fix.
It should be all right now.
Thx, Eric

On 09/01/2013 11:13, Ioan Eugen Stan wrote:

Hello Eric,

I don't have time to look over it now. I'm on it over the weekend. The
plugin should work for children pom's if we configure it in the
parent.

Cheers,

On Tue, Jan 8, 2013 at 6:54 PM, Eric Charles e...@apache.org wrote:

Yes sure, I had added a comment on the JIRA in that way.
I tough it was solving the following issue, but it doesn't. I will revert.

I see james-server-queue-jms pom which has extra definition
(maven-jar-plugin with goal test-jar, see  [3]) but not
james-server-protocols-smtp, resulting in the absence of test-jar for [2]
(well available in [1]).

I would be better to solve this for all module so we don't have to define in
all pom extra configuration for maven-jar-plugin.

Any idea to solve this as the apache parent and my last commit don't give
solution?

Thx, Eric

[1]
https://repository.apache.org/content/groups/snapshots/org/apache/james/james-server-queue-jms/3.0-beta5-SNAPSHOT/

[2]
https://repository.apache.org/content/groups/snapshots/org/apache/james/james-server-protocols-smtp/3.0-beta5-SNAPSHOT/


[3]
 build
 plugins
 plugin
 groupIdorg.apache.maven.plugins/groupId
 artifactIdmaven-jar-plugin/artifactId
 executions
 execution
 goals
 goaltest-jar/goal
 /goals
 /execution
 /executions
 /plugin
 /plugins
 /build

On 08/01/2013 16:41, Ioan Eugen Stan wrote:


Hello Eric,

I think most of the plugins are configured in apache pom and we should
avoid duplication as much as possible.

Cheers,

On Tue, Jan 8, 2013 at 4:18 PM,  e...@apache.org wrote:


Author: eric
Date: Tue Jan  8 14:18:13 2013
New Revision: 1430296

URL: http://svn.apache.org/viewvc?rev=1430296view=rev
Log:
James parent should enforce the test-jar generation (JAMES-1465)

Modified:
  james/project/trunk/pom.xml

Modified: james/project/trunk/pom.xml
URL:
http://svn.apache.org/viewvc/james/project/trunk/pom.xml?rev=1430296r1=1430295r2=1430296view=diff

==
--- james/project/trunk/pom.xml (original)
+++ james/project/trunk/pom.xml Tue Jan  8 14:18:13 2013
@@ -646,6 +646,38 @@
   groupIdorg.apache.maven.plugins/groupId
   artifactIdmaven-jar-plugin/artifactId
   version2.4/version
+executions
+  execution
+idjar/id
+phasepackage/phase
+goals
+  goaljar/goal
+/goals
+configuration
+  archive
+manifest
+
addDefaultSpecificationEntriestrue/addDefaultSpecificationEntries
+
addDefaultImplementationEntriestrue/addDefaultImplementationEntries
+/manifest
+  /archive
+/configuration
+  /execution
+  execution
+idtest-jar/id
+phasepackage/phase
+goals
+  goaltest-jar/goal
+/goals
+configuration
+  archive
+manifest
+
addDefaultSpecificationEntriestrue/addDefaultSpecificationEntries
+
addDefaultImplementationEntriestrue/addDefaultImplementationEntries
+/manifest
+  /archive
+/configuration
+  /execution
+   /executions
   /plugin
   plugin
   groupIdorg.apache.maven.plugins/groupId
@@ -740,6 +772,15 @@
   groupIdorg.apache.maven.plugins/groupId
   artifactIdmaven-source-plugin/artifactId
   version2.2.1/version
+executions
+  execution
+idattach-sources/id
+goals
+  goaljar/goal
+  goaltest-jar/goal
+/goals
+  /execution
+/executions
   /plugin
   plugin
   groupIdorg.apache.maven.plugins/groupId



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org







-
To unsubscribe, e-mail: server-dev-unsubscr

[jira] [Created] (JAMES-1467) Upgrade to mailet 2.5.1-SNAPSHOT

2013-01-09 Thread Eric Charles (JIRA)

Eric Charles created JAMES-1467:
---

 Summary: Upgrade to mailet 2.5.1-SNAPSHOT 
 Key: JAMES-1467
 URL: https://issues.apache.org/jira/browse/JAMES-1467
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles


This is needed to have the mailetcontainer load the correct package.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Resolved] (JAMES-1467) Upgrade to mailet 2.5.1-SNAPSHOT

2013-01-09 Thread Eric Charles (JIRA)


 [ 
https://issues.apache.org/jira/browse/JAMES-1467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles resolved JAMES-1467.
-

Resolution: Fixed

 Upgrade to mailet 2.5.1-SNAPSHOT 
 -

 Key: JAMES-1467
 URL: https://issues.apache.org/jira/browse/JAMES-1467
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles

 This is needed to have the mailetcontainer load the correct package.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Build failure on James trunk

2013-01-09 Thread Eric Charles


Hi Jerry,

The mailing list strips attachments.
Moreover, to submit content (code, doc...), it's best to open a JIRA and 
attach there the patch granting ASF for rights.


Thx, Eric

On 10/01/2013 06:55, Jerry Tian wrote:

Hi, Eric,

The document patch generated from SVN diff is attached.


--
Jerry Tian
Sent with Sparrow http://www.sparrowmailapp.com/?sig

On Wednesday, January 9, 2013 at 4:17 PM, Eric Charles wrote:


You're more than welcome :)
Feel free to submit the patch, I will happily commit it.

Thx, Eric


On 09/01/2013 07:46, Jerry Tian wrote:

cheers, finally catch it.

btw, if it helps to fix the document, please let me know. I found
that they are in server/src/site/xdoc/dev-build.xml , so if needed,
I can submit a patch of it.

Thanks.


-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
mailto:server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org
mailto:server-dev-h...@james.apache.org





-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Resolved] (JSIEVE-95) Upgrade to mailet 2.5.1-SNAPSHOT


 [ 
https://issues.apache.org/jira/browse/JSIEVE-95?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles resolved JSIEVE-95.


Resolution: Fixed

Was needed because 2.5.0 SNAPSHOT was not deployed.

 Upgrade to mailet 2.5.1-SNAPSHOT
 

 Key: JSIEVE-95
 URL: https://issues.apache.org/jira/browse/JSIEVE-95
 Project: James jSieve
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (MAILBOX-190) Add hbase jar to mailbox-integration-tester to fix compilation issue

Eric Charles created MAILBOX-190:


 Summary: Add hbase jar to mailbox-integration-tester to fix 
compilation issue
 Key: MAILBOX-190
 URL: https://issues.apache.org/jira/browse/MAILBOX-190
 Project: James Mailbox
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles


We need the jar upon the test-jar.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Resolved] (MAILBOX-190) Add hbase jar to mailbox-integration-tester to fix compilation issue


 [ 
https://issues.apache.org/jira/browse/MAILBOX-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles resolved MAILBOX-190.
--

Resolution: Fixed

committed

 Add hbase jar to mailbox-integration-tester to fix compilation issue
 

 Key: MAILBOX-190
 URL: https://issues.apache.org/jira/browse/MAILBOX-190
 Project: James Mailbox
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles

 We need the jar upon the test-jar.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Build failure on James trunk

2013-01-08 Thread Eric Charles


Hi Jerry,

I have update the jsieve pom, see JSIEVE-95, the 2.5.0-SNAPSHOT was not 
deployed in nexus (2.5.1-SNAPSHOT well).


The mailbox-integration-tester was compiling fine a few days ago here, 
but lamentally fails today with the same exception as yours. I have 
committed a fix (MAILBOX-190). Strange it has suddenly failed #mavenvoodoo


Could you svn up and retry?

Thx a lot for reporting,

Eric


On 08/01/2013 09:46, Jerry Tian wrote:

Hello,

When attempted to build from trunk and follow the latest development of James, 
I am stuck with two building errors. One is module version related, one is 
compiling error.

I am following the building instruction there:

http://james.apache.org/server/3/dev-build.html

After checking out, the repository has the following SVN meta:
***
$svn info
Path: .
URL: https://svn.apache.org/repos/asf/james/current
Repository Root: https://svn.apache.org/repos/asf
Repository UUID: 13f79535-47bb-0310-9956-ffa450edef68
Revision: 1430166
Node Kind: directory
Schedule: normal
Last Changed Author: eric
Last Changed Rev: 1426727
Last Changed Date: 2012-12-29 17:23:19 +0800 (Sat, 29 Dec 2012)

***

The first problem is JSieve's version mismatch, I have changed it, or maven 
will complain about missing dependencies, stop further building. A SVN diff 
will clearly show my changes:

***
$svn diff jsieve/pom.xml
Index: jsieve/pom.xml
===
--- jsieve/pom.xml (revision 1430169)
+++ jsieve/pom.xml (working copy)
@@ -79,7 +79,7 @@
  project.build.sourceEncodingUTF-8/project.build.sourceEncoding

  mime4j.version0.7.2/mime4j.version
-apache-mailet.version2.5.0-SNAPSHOT/apache-mailet.version
+apache-mailet.version2.5.1-SNAPSHOT/apache-mailet.version
  junit.version4.10/junit.version
  jmock.version1.2.0/jmock.version
  log4j.version1.2.14/log4j.version



***

The second problem is beyond my knowledge, I can only dump the maven output 
here.

***

[INFO] 
[INFO] Building Apache James Mailbox Integration Tester 1.8.1
[INFO] 
..
[INFO]
[INFO] --- maven-clean-plugin:2.4.1:clean (default-clean) @ 
apache-james-mailbox-integration-tester ---
[INFO]
[INFO] --- maven-remote-resources-plugin:1.2.1:process (default) @ 
apache-james-mailbox-integration-tester ---
[WARNING] Invalid POM for 
org.apache.james:apache-james-mailbox-hbase:jar:0.5-SNAPSHOT, transitive 
dependencies (if any) will not be available, enable debug logging for more 
details
[WARNING] Invalid project model for artifact 
[apache-james-mailbox-hbase:org.apache.james:0.5-SNAPSHOT]. It will be ignored 
by the remote resources Mojo.
[INFO]
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
apache-james-mailbox-integration-tester ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 113 resources
[INFO] Copying 3 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ 
apache-james-mailbox-integration-tester ---
[INFO] Nothing to compile - all classes are up to date
[INFO]
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
apache-james-mailbox-integration-tester ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:testCompile (default-testCompile) @ 
apache-james-mailbox-integration-tester ---
[INFO] Compiling 95 source files to 
/Users/jerry/Tmp/apache-james-trunk/mailbox-integration-tester/target/test-classes
[INFO] -
[ERROR] COMPILATION ERROR :
[INFO] -
[ERROR] 
/Users/jerry/Tmp/apache-james-trunk/mailbox-integration-tester/src/test/java/org/apache/james/mailbox/hbase/host/HBaseHostSystem.java:[25,30]
 cannot find symbol
symbol  : class HBaseConfiguration
location: package org.apache.hadoop.hbase
[ERROR] 
/Users/jerry/Tmp/apache-james-trunk/mailbox-integration-tester/src/test/java/org/apache/james/mailbox/hbase/host/HBaseHostSystem.java:[25,30]
 cannot find symbol
symbol  : class HBaseConfiguration
location: package org.apache.hadoop.hbase
[ERROR] 
/Users/jerry/Tmp/apache-james-trunk/mailbox-integration-tester/src/test/java/org/apache/james/mailbox/hbase/host/HBaseHostSystem.java:[81,19]
 cannot find symbol
symbol  : variable HBaseConfiguration
location: class org.apache.james.mailbox.hbase.host.HBaseHostSystem
[INFO] 3 errors
[INFO] -
[INFO]

[jira] [Closed] (MAILBOX-190) Add hbase jar to mailbox-integration-tester to fix compilation issue


 [ 
https://issues.apache.org/jira/browse/MAILBOX-190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed MAILBOX-190.



 Add hbase jar to mailbox-integration-tester to fix compilation issue
 

 Key: MAILBOX-190
 URL: https://issues.apache.org/jira/browse/MAILBOX-190
 Project: James Mailbox
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles

 We need the jar upon the test-jar.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JSIEVE-95) Upgrade to mailet 2.5.1-SNAPSHOT


 [ 
https://issues.apache.org/jira/browse/JSIEVE-95?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JSIEVE-95.
--


 Upgrade to mailet 2.5.1-SNAPSHOT
 

 Key: JSIEVE-95
 URL: https://issues.apache.org/jira/browse/JSIEVE-95
 Project: James jSieve
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: Build failure on James trunk

2013-01-08 Thread Eric Charles

There may have an configuration to tune regarding the deployement of 
test-jar.


In the meantime, could you try to build with mvn install instead of 
mvn compile or mvn package (I assume you don't invoke install, 
otherwise you shouldn't have that issue).


Thx in advance for your feedback,

Eric

On 08/01/2013 11:07, Jerry Tian wrote:

Hi, Eric,

Just updated the repo and made a clean build, it passed the previous modules, 
but failed on another one.

Here is the output.

[INFO]
[INFO] 
[INFO] Building Apache James Postage 0.1-SNAPSHOT
[INFO] 
Downloading: 
http://repository.apache.org/snapshots/org/apache/james/james-server-protocols-smtp/3.0-beta5-SNAPSHOT/maven-metadata.xml
Downloaded: 
http://repository.apache.org/snapshots/org/apache/james/james-server-protocols-smtp/3.0-beta5-SNAPSHOT/maven-metadata.xml
 (811 B at 0.5 KB/sec)
Downloading: 
http://repo.maven.apache.org/maven2/commons-pool/commons-pool/1.5.4/commons-pool-1.5.4.jar
Downloaded: 
http://repo.maven.apache.org/maven2/commons-pool/commons-pool/1.5.4/commons-pool-1.5.4.jar
 (94 KB at 59.3 KB/sec)
Downloading: 
http://repository.apache.org/snapshots/org/apache/james/james-server-protocols-smtp/3.0-beta5-SNAPSHOT/james-server-protocols-smtp-3.0-beta5-SNAPSHOT-tests.jar
[INFO] 
[INFO] Reactor Summary:
[INFO]
[INFO] Apache James Project .. SUCCESS [0.762s]
[INFO] Apache James :: Server  SUCCESS [0.444s]
[INFO] Apache James :: Server :: Data  :: API  SUCCESS [1.245s]
[INFO] Apache James :: Server :: Lifecycle API ... SUCCESS [0.286s]
[INFO] Apache James :: JSieve  SUCCESS [0.021s]
[INFO] Apache James :: jSieve :: Manager . SUCCESS [0.018s]
[INFO] Apache James :: jSieve :: Manager API . SUCCESS [0.175s]
[INFO] Apache James :: Server :: Filesystem API .. SUCCESS [0.438s]
[INFO] Apache James :: Server :: Lifecycle Spring  SUCCESS [0.582s]
[INFO] Apache James :: Server :: Mailetcontainer API . SUCCESS [0.401s]
[INFO] Apache James :: Server :: Common Utilities  SUCCESS [0.262s]
[INFO] Apache James :: Protocols . SUCCESS [0.109s]
[INFO] Apache James :: Protocols :: API .. SUCCESS [0.396s]
[INFO] Apache James :: Protocols :: Netty Implementation . SUCCESS [0.353s]
[INFO] Apache James :: Server :: Protocols Library ... SUCCESS [0.490s]
[INFO] Apache James :: Server :: Core  SUCCESS [0.236s]
[INFO] Apache James :: Mailbox ... SUCCESS [0.230s]
[INFO] Apache James :: Mailbox :: API  SUCCESS [1.803s]
[INFO] Apache James :: Mailbox :: Store Framework  SUCCESS [1.803s]
[INFO] Apache James :: Mailbox :: In Memory .. SUCCESS [0.615s]
[INFO] Apache James :: Mailbox :: Tools .. SUCCESS [0.898s]
[INFO] Apache James :: Server :: Container Spring  SUCCESS [0.866s]
[INFO] Apache James :: Server :: Client .. SUCCESS [0.578s]
[INFO] Apache James :: Server :: DNS Service :: API .. SUCCESS [0.293s]
[INFO] Apache James :: Server :: DNS Service :: Library .. SUCCESS [0.229s]
[INFO] Apache James :: Server :: Mail Queue :: API ... SUCCESS [0.339s]
[INFO] Apache James :: Server :: Mailetcontainer Camel ... SUCCESS [0.443s]
[INFO] Apache James :: Mailbox :: Maildir  SUCCESS [0.673s]
[INFO] Apache James :: Server :: Mailbox Adapter . SUCCESS [0.342s]
[INFO] Apache James :: Server :: DNS Service :: Implementation  SUCCESS [0.244s]
[INFO] Apache James :: Server :: Data :: Library . SUCCESS [0.471s]
[INFO] Apache James :: Server :: Data :: LDAP Implementation  SUCCESS [0.213s]
[INFO] Apache James :: Protocols :: SMTP . SUCCESS [0.960s]
[INFO] Apache James :: Server :: SMTP  SUCCESS [0.624s]
[INFO] Apache James :: Protocols :: IMAP . SUCCESS [0.968s]
[INFO] Apache James :: Server :: IMAP  SUCCESS [0.369s]
[INFO] Apache James :: Protocols :: LMTP . SUCCESS [0.281s]
[INFO] Apache James :: Server :: LMTP  SUCCESS [0.530s]
[INFO] Apache James :: Protocols :: POP3 . SUCCESS [0.283s]
[INFO] Apache James :: Server :: POP3  SUCCESS [0.368s]
[INFO] Apache James :: Server :: FetchMail ... SUCCESS [0.250s]
[INFO] Apache James :: Server :: Mail Queue :: File .. SUCCESS [0.185s]
[INFO] Apache James :: Server :: Mail Queue :: JMS ... SUCCESS [0.555s]
[INFO] Apache James :: Server :: Mail Queue :: ActiveMQ .. SUCCESS [0.511s]
[INFO] Apache James :: jSieve :: Core  SUCCESS [0.500s]
[INFO] Apache James :: Mailets parent and aggregator .

[jira] [Created] (JAMES-1465) James parent should enforce the test-jar generation

Eric Charles created JAMES-1465:
---

 Summary: James parent should enforce the test-jar generation
 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1465) James parent should enforce the test-jar generation


 [ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1465.
---


 James parent should enforce the test-jar generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Reopened] (JAMES-1465) James parent should enforce the test-jar generation


 [ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles reopened JAMES-1465:
-


Doesn't seem to work.
Moreover, the added definition should already be there via the apache parent.

 James parent should enforce the test-jar generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Re: svn commit: r1430296 - /james/project/trunk/pom.xml

2013-01-08 Thread Eric Charles


Yes sure, I had added a comment on the JIRA in that way.
I tough it was solving the following issue, but it doesn't. I will revert.

I see james-server-queue-jms pom which has extra definition 
(maven-jar-plugin with goal test-jar, see  [3]) but not 
james-server-protocols-smtp, resulting in the absence of test-jar for 
[2] (well available in [1]).


I would be better to solve this for all module so we don't have to 
define in all pom extra configuration for maven-jar-plugin.


Any idea to solve this as the apache parent and my last commit don't 
give solution?


Thx, Eric

[1] 
https://repository.apache.org/content/groups/snapshots/org/apache/james/james-server-queue-jms/3.0-beta5-SNAPSHOT/


[2] 
https://repository.apache.org/content/groups/snapshots/org/apache/james/james-server-protocols-smtp/3.0-beta5-SNAPSHOT/



[3]
build
plugins
plugin
groupIdorg.apache.maven.plugins/groupId
artifactIdmaven-jar-plugin/artifactId
executions
execution
goals
goaltest-jar/goal
/goals
/execution
/executions
/plugin
/plugins
/build

On 08/01/2013 16:41, Ioan Eugen Stan wrote:

Hello Eric,

I think most of the plugins are configured in apache pom and we should
avoid duplication as much as possible.

Cheers,

On Tue, Jan 8, 2013 at 4:18 PM,  e...@apache.org wrote:

Author: eric
Date: Tue Jan  8 14:18:13 2013
New Revision: 1430296

URL: http://svn.apache.org/viewvc?rev=1430296view=rev
Log:
James parent should enforce the test-jar generation (JAMES-1465)

Modified:
 james/project/trunk/pom.xml

Modified: james/project/trunk/pom.xml
URL: 
http://svn.apache.org/viewvc/james/project/trunk/pom.xml?rev=1430296r1=1430295r2=1430296view=diff
==
--- james/project/trunk/pom.xml (original)
+++ james/project/trunk/pom.xml Tue Jan  8 14:18:13 2013
@@ -646,6 +646,38 @@
  groupIdorg.apache.maven.plugins/groupId
  artifactIdmaven-jar-plugin/artifactId
  version2.4/version
+executions
+  execution
+idjar/id
+phasepackage/phase
+goals
+  goaljar/goal
+/goals
+configuration
+  archive
+manifest
+  
addDefaultSpecificationEntriestrue/addDefaultSpecificationEntries
+  
addDefaultImplementationEntriestrue/addDefaultImplementationEntries
+/manifest
+  /archive
+/configuration
+  /execution
+  execution
+idtest-jar/id
+phasepackage/phase
+goals
+  goaltest-jar/goal
+/goals
+configuration
+  archive
+manifest
+  
addDefaultSpecificationEntriestrue/addDefaultSpecificationEntries
+  
addDefaultImplementationEntriestrue/addDefaultImplementationEntries
+/manifest
+  /archive
+/configuration
+  /execution
+   /executions
  /plugin
  plugin
  groupIdorg.apache.maven.plugins/groupId
@@ -740,6 +772,15 @@
  groupIdorg.apache.maven.plugins/groupId
  artifactIdmaven-source-plugin/artifactId
  version2.2.1/version
+executions
+  execution
+idattach-sources/id
+goals
+  goaljar/goal
+  goaltest-jar/goal
+/goals
+  /execution
+/executions
  /plugin
  plugin
  groupIdorg.apache.maven.plugins/groupId



-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org







-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-1465) James parent should enforce the test-jar and source jars generation


[ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13547683#comment-13547683
 ] 

Eric Charles commented on JAMES-1465:
-

Finally, not in apache-parent.
Terry identified we should put the definitions under /project/build/plugins.

 James parent should enforce the test-jar and source jars generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Updated] (JAMES-1465) James parent should enforce the test-jar and source jars generation


 [ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles updated JAMES-1465:


Summary: James parent should enforce the test-jar and source jars 
generation  (was: James parent should enforce the test-jar generation)

 James parent should enforce the test-jar and source jars generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Resolved] (JAMES-1465) James parent should enforce the test-jar and source jars generation


 [ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles resolved JAMES-1465.
-

   Resolution: Fixed
Fix Version/s: 3.0-beta5

committed.

 James parent should enforce the test-jar and source jars generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1465) James parent should enforce the test-jar and source jars generation


 [ 
https://issues.apache.org/jira/browse/JAMES-1465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles closed JAMES-1465.
---


 James parent should enforce the test-jar and source jars generation
 ---

 Key: JAMES-1465
 URL: https://issues.apache.org/jira/browse/JAMES-1465
 Project: James Server
  Issue Type: Bug
Reporter: Eric Charles
Assignee: Eric Charles
 Fix For: 3.0-beta5




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Created] (JAMES-1466) Bump james-project version to 1.8.2-SNAPSHOT

Eric Charles created JAMES-1466:
---

 Summary: Bump james-project version to 1.8.2-SNAPSHOT
 Key: JAMES-1466
 URL: https://issues.apache.org/jira/browse/JAMES-1466
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Resolved] (JAMES-1466) Bump james-project version to 1.8.2-SNAPSHOT


 [ 
https://issues.apache.org/jira/browse/JAMES-1466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eric Charles resolved JAMES-1466.
-

Resolution: Fixed

 Bump james-project version to 1.8.2-SNAPSHOT
 

 Key: JAMES-1466
 URL: https://issues.apache.org/jira/browse/JAMES-1466
 Project: James Server
  Issue Type: Bug
Affects Versions: 3.0-beta5
Reporter: Eric Charles
Assignee: Eric Charles



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Closed] (JAMES-1466) Bump james-project version to 1.8.2-SNAPSHOT