Replace MINA with Netty
Norman Maurer
I heared you are interested to replace your current MINA code with
Netty. Is this still the case ? If so I would like to volunteer to
work on it ;)
Just let me know and I will fork the code base and start working.
Thanks,
Norman
Rustam Aliyev
That would be great!
I'll prepare branch today which will include Netty bundles. Also I was planning to put together one page doc for setting up dev environment - should help to get started quickly.
Many thanks,
Rustam.
Norman Maurer
Bye,
Norman
Rustam Aliyev
I have done some investigation for what may be needed on Cassandra side for POP3/IMAP. Shouldn't be too complicated.
Also, right now are now in the middle of migration - moving 150K subs with 20-30M messages to ElasticInbox. So it may seem that development slowed down a bit for a couple of weeks. But we will continue as soon as we done with migration.
--
Rustam.
Norman Maurer
Rustam Aliyev
Just added "netty" branch. Netty is now dependency for LMTP module.
Also, some dev environment notes are here:
https://github.com/elasticinbox/elasticinbox/wiki/Development-environment
--
Rustam.
On Fri Dec 16 13:04:08 2011, Norman Maurer wrote:
> Ok then let we work on integration of protocols with netty under the
> hood. This will also make it easy to add smtp and pop3.
>
> Bye,
> Norman
>
>
> 2011/12/16 Rustam Aliyev <rus...@code.az <mailto:rus...@code.az>>
>
> That would be great Norman!
>
> I have done some investigation for what may be needed on Cassandra
> side for POP3/IMAP. Shouldn't be too complicated.
>
> Also, right now are now in the middle of migration - moving 150K
> subs with 20-30M messages to ElasticInbox. So it may seem that
> development slowed down a bit for a couple of weeks. But we will
> continue as soon as we done with migration.
>
> --
> Rustam.
>
> On 16/12/2011 12:54, Norman Maurer wrote:
>> I may also be able to expieriement a bit with using james
>> protocols which will make it more easy to change the transport
>> impl later. WDYT ?
>>
>> Bye,
>> Norman
>>
>>
>> 2011/12/16 Rustam Aliyev <rus...@code.az <mailto:rus...@code.az>>
Norman Maurer
Bye,
Norman
Norman Maurer
you can now find the first "version" of the lmtp module which use
Apache James Protocols. This allows easy changing of the transport
implementation. Apache James Protocols comes with a Netty based
transport which the lmtp module now use. Please have a look and let me
know what you thing ;)
I think there is room for improvements but I think you can see that
your lmtp code is now a lot smaller ..
https://github.com/normanmaurer/elasticinbox/tree/netty
You can fire it up via osgi as the old version.
Bye,
Norman
On 17 Dez., 20:50, Norman Maurer <norman.mau...@googlemail.com> wrote:
> Thanks I will have a look. I think I will be able to get something working
> within the next week. So stay tuned ...
>
> Bye,
> Norman
>
> 2011/12/17 Rustam Aliyev <rus...@code.az>
>
>
>
>
>
>
>
> > Hi Norman,
>
> > Just added "netty" branch. Netty is now dependency for LMTP module. Also,
> > some dev environment notes are here:
>
> >https://github.com/**elasticinbox/elasticinbox/**
> > wiki/Development-environment<https://github.com/elasticinbox/elasticinbox/wiki/Development-environ...>
Rustam Aliyev
That's great! Thanks. I was actually checking out your changes
yesterday and noticed that you chopped off a lot of stuff :). I
personally like it. I will have a closer look later this week.
One thing I noticed is that you removed extended LMTP/SMTP response
status codes and just use basic ones. We don't rely on them, but I
think there are some systems that do. What is your experience?
Also, I wanted to add LMTP testing to integration tests. Do you have
some simple code or client LMTP library which we could use for it?
Many Thanks,
Rustam.
Norman Maurer
I thought I have ENHANCEDSTATUSCODES in there as we use them in Apache James Protocols also.. I will review again to see if its the case ;)
For LMTP testing I can suggest you niosmtp (which I wrote =P). Its a fully async SMTP / LMTP Client which uses netty under the hood.
Bye,
Norman
Rustam Aliyev
I had reviewed it today, looks really good. I had to switch to
protocols RC1 though. There was some issue with RC2-SNAPSHOT logging
(3rd parameter for SMTPProtocol was required here:
https://github.com/normanmaurer/elasticinbox/blob/netty/modules/lmtp/src/main/java/com/elasticinbox/lmtp/LMTPProxyServer.java#L56).
Can you please send pull request for netty branch? I will clean it up a
bit and merge to master as soon as protocols 1.6 final will be released.
Thanks for your effort and have a good holidays!
--
Rustam.
On Tue Dec 20 18:31:15 2011, Norman Maurer wrote:
> Hi there,
>
> I thought I have ENHANCEDSTATUSCODES in there as we use them in Apache
> James Protocols also.. I will review again to see if its the case ;)
>
> For LMTP testing I can suggest you niosmtp (which I wrote =P). Its a
> fully async SMTP / LMTP Client which uses netty under the hood.
>
> Bye,
> Norman
>
>
> 2011/12/20 Rustam Aliyev <rus...@code.az <mailto:rus...@code.az>>
>
> Hi Norman,
>
> That's great! Thanks. I was actually checking out your changes
> yesterday and noticed that you chopped off a lot of stuff :). I
> personally like it. I will have a closer look later this week.
>
> One thing I noticed is that you removed extended LMTP/SMTP
> response status codes and just use basic ones. We don't rely on
> them, but I think there are some systems that do. What is your
> experience?
>
> Also, I wanted to add LMTP testing to integration tests. Do you
> have some simple code or client LMTP library which we could use
> for it?
>
> Many Thanks,
> Rustam.
>
>
>
> On Tue Dec 20 17:50:44 2011, Norman Maurer wrote:
>
> Hi there,
>
> you can now find the first "version" of the lmtp module which use
> Apache James Protocols. This allows easy changing of the transport
> implementation. Apache James Protocols comes with a Netty based
> transport which the lmtp module now use. Please have a look
> and let me
> know what you thing ;)
>
> I think there is room for improvements but I think you can see
> that
> your lmtp code is now a lot smaller ..
>
> https://github.com/__normanmaurer/elasticinbox/__tree/netty
> <https://github.com/normanmaurer/elasticinbox/tree/netty>
>
> You can fire it up via osgi as the old version.
>
> Bye,
> Norman
>
> On 17 Dez., 20:50, Norman
> Maurer<norman.mau...@__googlemail.com
> <mailto:norman.mau...@googlemail.com>> wrote:
>
> Thanks I will have a look. I think I will be able to get
> something working
> within the next week. So stay tuned ...
>
> Bye,
> Norman
>
> 2011/12/17 Rustam Aliyev<rus...@code.az
> <mailto:rus...@code.az>>
>
>
>
>
>
>
>
> Hi Norman,
>
>
> Just added "netty" branch. Netty is now dependency for
> LMTP module. Also,
> some dev environment notes are here:
>
>
> https://github.com/**__elasticinbox/elasticinbox/**
> <https://github.com/**elasticinbox/elasticinbox/**>
> wiki/Development-environment<h__ttps://github.com/__elasticinbox/elasticinbox/__wiki/Development-environ..
> <https://github.com/elasticinbox/elasticinbox/wiki/Development-environ..>.>
>
>
> --
> Rustam.
>
>
> On Fri Dec 16 13:04:08 2011, Norman Maurer wrote:
>
>
> Ok then let we work on integration of protocols
> with netty under the
> hood. This will also make it easy to add smtp and
> pop3.
>
>
> Bye,
> Norman
>
>
> 2011/12/16 Rustam Aliyev<rus...@code.az
> <mailto:rus...@code.az><mailto:r__us...@code.az
> <mailto:rus...@code.az>>>
>
>
> That would be great Norman!
>
>
> I have done some investigation for what may be
> needed on Cassandra
> side for POP3/IMAP. Shouldn't be too complicated.
>
>
> Also, right now are now in the middle of
> migration - moving 150K
> subs with 20-30M messages to ElasticInbox. So
> it may seem that
> development slowed down a bit for a couple of
> weeks. But we will
> continue as soon as we done with migration.
>
>
> --
> Rustam.
>
>
> On 16/12/2011 12:54, Norman Maurer wrote:
>
>
> I may also be able to expieriement a bit
> with using james
> protocols which will make it more easy to
> change the transport
> impl later. WDYT ?
>
>
> Bye,
> Norman
>
>
> 2011/12/16 Rustam Aliyev<rus...@code.az
> <mailto:rus...@code.az><mailto:r__us...@code.az <mailto:rus...@code.az>>>
Norman Maurer
yeah the RC2-SNAPSHOT is under heavy development atm. I'm currently integrating IMAP directly, so once I'm done it will be really easy to add it. So for now I will use RC1.
I will also add some more changes and then issue the pull request..
Bye,
Norman
Rustam Aliyev
I was cleaning up stuff after merge and noticed that you are using
Geronimo implementation of JavaMail. Is it safe to replace it with
Sun's implementation? We are already using Sun JavaMail in other parts.
Thanks,
Rustam.
On Fri Dec 30 08:38:24 2011, Norman Maurer wrote:
> Hi there,
>
> yeah the RC2-SNAPSHOT is under heavy development atm. I'm currently
> integrating IMAP directly, so once I'm done it will be really easy to
> add it. So for now I will use RC1.
>
> I will also add some more changes and then issue the pull request..
>
> Bye,
> Norman
>
>
> 2011/12/29 Rustam Aliyev <rus...@code.az <mailto:rus...@code.az>>
>
> Hi Norman,
>
> I had reviewed it today, looks really good. I had to switch to
> protocols RC1 though. There was some issue with RC2-SNAPSHOT
> logging (3rd parameter for SMTPProtocol was required here:
> https://github.com/__normanmaurer/elasticinbox/__blob/netty/modules/lmtp/src/__main/java/com/elasticinbox/__lmtp/LMTPProxyServer.java#L56
> <https://github.com/normanmaurer/elasticinbox/blob/netty/modules/lmtp/src/main/java/com/elasticinbox/lmtp/LMTPProxyServer.java#L56>)__.
>
> Can you please send pull request for netty branch? I will clean it
> up a bit and merge to master as soon as protocols 1.6 final will
> be released.
>
> Thanks for your effort and have a good holidays!
>
> --
> Rustam.
>
>
> On Tue Dec 20 18:31:15 2011, Norman Maurer wrote:
>
> Hi there,
>
> I thought I have ENHANCEDSTATUSCODES in there as we use them
> in Apache James Protocols also.. I will review again to see if
> its the case ;)
>
> For LMTP testing I can suggest you niosmtp (which I wrote =P).
> Its a fully async SMTP / LMTP Client which uses netty under
> the hood.
>
> Bye,
> Norman
>
>
> 2011/12/20 Rustam Aliyev <rus...@code.az
> <mailto:rus...@code.az> <mailto:rus...@code.az
> https://github.com/____normanmaurer/elasticinbox/____tree/netty <https://github.com/__normanmaurer/elasticinbox/__tree/netty>
>
> <https://github.com/__normanmaurer/elasticinbox/__tree/netty
> <https://github.com/normanmaurer/elasticinbox/tree/netty>>
>
> You can fire it up via osgi as the old version.
>
> Bye,
> Norman
>
> On 17 Dez., 20:50, Norman
> Maurer<norman.mau...@__googlem__ail.com
> <http://googlemail.com>
> <mailto:norman.mau...@__googlemail.com
> <mailto:norman.mau...@googlemail.com>>> wrote:
>
>
> Thanks I will have a look. I think I will be able
> to get
> something working
> within the next week. So stay tuned ...
>
> Bye,
> Norman
>
> 2011/12/17 Rustam Aliyev<rus...@code.az
> <mailto:rus...@code.az>
> <mailto:rus...@code.az <mailto:rus...@code.az>>>
>
>
>
>
>
>
>
>
> Hi Norman,
>
>
> Just added "netty" branch. Netty is now
> dependency for
> LMTP module. Also,
> some dev environment notes are here:
>
>
> https://github.com/**____elasticinbox/elasticinbox/**
> <https://github.com/**__elasticinbox/elasticinbox/**>
> <https://github.com/**__elasticinbox/elasticinbox/**
> <https://github.com/**elasticinbox/elasticinbox/**>>
>
> wiki/Development-environment<__h__ttps://github.com/____elasticinbox/elasticinbox/____wiki/Development-environ
> <http://github.com/__elasticinbox/elasticinbox/__wiki/Development-environ>..
> <https://github.com/__elasticinbox/elasticinbox/__wiki/Development-environ.
> <https://github.com/elasticinbox/elasticinbox/wiki/Development-environ.>.>.>
>
>
>
> --
> Rustam.
>
>
> On Fri Dec 16 13:04:08 2011, Norman Maurer wrote:
>
>
> Ok then let we work on integration of protocols
> with netty under the
> hood. This will also make it easy to add
> smtp and
> pop3.
>
>
> Bye,
> Norman
>
>
> 2011/12/16 Rustam Aliyev<rus...@code.az
> <mailto:rus...@code.az>
> <mailto:rus...@code.az
> <mailto:rus...@code.az>><__mailto:r__us...@code.az
> <mailto:r__us...@code.az>
>
> <mailto:rus...@code.az <mailto:rus...@code.az>>>>
> <mailto:rus...@code.az
> <mailto:rus...@code.az>><__mailto:r__us...@code.az
> <mailto:r__us...@code.az> <mailto:rus...@code.az
> <mailto:rus...@code.az>>>>
>
>
>
> Hi Norman,
>
>
Norman Maurer
Norman Maurer
Rustam Aliyev
Regarding license... well, to be honest didn't have time to do enough
research. GPL was chosen purely to be on a safe side for now. I'll
update our email conversation on license as soon as we decide.
--
Rustam.
>>> Maurer<norman.mau...@__googlem__ail.com <http://__googlem__ail.com>
>>> <http://googlemail.com>
>>> <mailto:norman.mau...@__googlemail.com <http://__googlemail.com>
>>> <mailto:norman.mau...@googlemail.com <http://googlemail.com>>>> wrote:
>>>
>>>
>>> Thanks I will have a look. I think I will be able
>>> to get
>>> something working
>>> within the next week. So stay tuned ...
>>>
>>> Bye,
>>> Norman
>>>
>>> 2011/12/17 Rustam Aliyev<rus...@code.az
>>> <mailto:rus...@code.az>
>>> <mailto:rus...@code.az <mailto:rus...@code.az>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> Hi Norman,
>>>
>>>
>>> Just added "netty" branch. Netty is now
>>> dependency for
>>> LMTP module. Also,
>>> some dev environment notes are here:
>>>
>>>
>>> https://github.com/**____elasticinbox/elasticinbox/**
>>> <https://github.com/**__elasticinbox/elasticinbox/**>
>>> <https://github.com/**__elasticinbox/elasticinbox/**
>>> <https://github.com/**elasticinbox/elasticinbox/**>>
>>> wiki/Development-environment<__h__ttps://github.com/____elasticinbox/elasticinbox/____wiki/Development-environ
>>> <http://github.com/____elasticinbox/elasticinbox/____wiki/Development-environ>
Norman Maurer
Norman Maurer