You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by Robert Burrell Donkin <ro...@gmail.com> on 2008/08/03 23:40:06 UTC
Re: svn commit: r682219 - in /james/server/trunk: core-library/src/main/java/org/apache/james/core/ experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/ experimental-seda-imap-function/src/test/java/org/apache/james
On Sun, Aug 3, 2008 at 10:28 PM, Stefano Bagnara <ap...@bago.org> wrote:
> rdonkin@apache.org ha scritto:
>>
>> Author: rdonkin
>> Date: Sun Aug 3 14:21:45 2008
>> New Revision: 682219
>>
>> URL: http://svn.apache.org/viewvc?rev=682219&view=rev
>> Log:
>> Concurrency tests for IMAP
>
>> [...]
>>
>> Modified:
>>
>> james/server/trunk/core-library/src/main/java/org/apache/james/core/MailImpl.java
>> [...]
>
> This commit included the changes you did to MailImpl working on JAMES-850. I
> don't know if this is by purpose or by mistake, but I'm not sure they are
> good in the long term unless there is a need (unless they fix the issue,
> because the "full clone" on MailImpl was introduced for performance issues).
i spotted this almost as soon as i commit it :-/
should be reverted now
- robert
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org
Re: svn commit: r682219 - in /james/server/trunk: core-library/src/main/java/org/apache/james/core/
experimental-seda-imap-function/src/main/java/org/apache/james/experimental/imapserver/
experimental-seda-imap-function/src/test/java/org/apache/james
Posted by Stefano Bagnara <ap...@bago.org>.
Robert Burrell Donkin ha scritto:
> On Sun, Aug 3, 2008 at 10:28 PM, Stefano Bagnara <ap...@bago.org> wrote:
>> rdonkin@apache.org ha scritto:
>>> Author: rdonkin
>>> Date: Sun Aug 3 14:21:45 2008
>>> New Revision: 682219
>>>
>>> URL: http://svn.apache.org/viewvc?rev=682219&view=rev
>>> Log:
>>> Concurrency tests for IMAP
>>> [...]
>>>
>>> Modified:
>>>
>>> james/server/trunk/core-library/src/main/java/org/apache/james/core/MailImpl.java
>>> [...]
>> This commit included the changes you did to MailImpl working on JAMES-850. I
>> don't know if this is by purpose or by mistake, but I'm not sure they are
>> good in the long term unless there is a need (unless they fix the issue,
>> because the "full clone" on MailImpl was introduced for performance issues).
>
> i spotted this almost as soon as i commit it :-/
>
> should be reverted now
No problem, I'm happy it was a mistake ;-)
For the record if it helps you can even commit it in the short term, but
I wanted to point out that in that case we'll have to discuss what to do
with it in the long term.
Stefano
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org