You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mina.apache.org by Trustin Lee <tr...@gmail.com> on 2007/09/27 00:24:45 UTC
[VOTE] Import KeepAlive functionality contribution under MINA PMC
Hi again.
There's an important contribution from Eero Nevalainen that will help
users to implement sending a protocol-specific keep-alive message
periodically:
https://issues.apache.org/jira/browse/DIRMINA-354
It's just a few classes, so I thought we could just import the code,
but procedure is procedure. :)
Software grant has been received long time ago, and also again, it was
not processed on time due to my laziness. There's no reason not to
import these classes IMO; it's just a few classes that makes people's
life a lot easier.
Once vote passes, I will start the paperwork and all the required procedure.
[ ]: +1, import
[ ] 0, abstain
[ ]: -1, don't import
Cheers,
Trustin
--
what we call human nature is actually human habit
--
http://gleamynode.net/
--
PGP Key ID: 0x0255ECA6
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Maarten Bosteels <mb...@gmail.com>.
[x]: +1, import
Re: [VOTE] Import KeepAlive functionality contribution under MINA
PMC
Posted by Mike Heath <mh...@apache.org>.
Trustin Lee wrote:
> [x]: +1, import
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Emmanuel Lecharny <el...@gmail.com>.
> [X]: +1, import
--
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Alex Karasulu <ak...@apache.org>.
>
> [X]: +1, import
Alex
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Trustin Lee <tr...@gmail.com>.
Now let me close the vote. Sorry for being late!
Trustin
On 10/16/07, mat <fo...@gmail.com> wrote:
> +1 very important (Definitely a MUST filter)
>
> On 9/27/07, Mark <el...@gmail.com> wrote:
> >
> > [X]: +1, import
> >
> > On 9/26/07, Michael Grundvig <mi...@electrotank.com> wrote:
> > >
> > > +1, import
> > >
> > > ----- Original Message -----
> > > From: "Trustin Lee" <tr...@gmail.com>
> > > To: <de...@mina.apache.org>
> > > Sent: Wednesday, September 26, 2007 5:24 PM
> > > Subject: [VOTE] Import KeepAlive functionality contribution under MINA
> > PMC
> > >
> > >
> > > > Hi again.
> > > >
> > > > There's an important contribution from Eero Nevalainen that will help
> > > > users to implement sending a protocol-specific keep-alive message
> > > > periodically:
> > > >
> > > > https://issues.apache.org/jira/browse/DIRMINA-354
> > > >
> > > > It's just a few classes, so I thought we could just import the code,
> > > > but procedure is procedure. :)
> > > >
> > > > Software grant has been received long time ago, and also again, it was
> > > > not processed on time due to my laziness. There's no reason not to
> > > > import these classes IMO; it's just a few classes that makes people's
> > > > life a lot easier.
> > > >
> > > > Once vote passes, I will start the paperwork and all the required
> > > > procedure.
> > > >
> > > > [ ]: +1, import
> > > > [ ] 0, abstain
> > > > [ ]: -1, don't import
> > > >
> > > > Cheers,
> > > > Trustin
> > > > --
> > > > what we call human nature is actually human habit
> > > > --
> > > > http://gleamynode.net/
> > > > --
> > > > PGP Key ID: 0x0255ECA6
> > >
> > >
> >
> >
> > --
> > ..Cheers
> > Mark
> >
>
--
what we call human nature is actually human habit
--
http://gleamynode.net/
--
PGP Key ID: 0x0255ECA6
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by mat <fo...@gmail.com>.
Should your ActiveKeepAliveFilter, assiveKeepAliveFilter both extend
IoFilterAdapter like in the tutorial "Writing IoFilter"? And you can return
IoFilter in Factory method?
On 10/16/07, mat <fo...@gmail.com> wrote:
>
> Several questions on your KeepAliveFilter.
>
> 1) Why createActiveKeepAliveFilter, createPassiveKeepAliveFilter NOT
> return IoFilter(A factory method is better to return the interface instead
> of concrete class)? And my server or client has to extends ChainedIoHandler,
> otherwises, i can't use " config.getFilterChain().addFirst" in my original
> code.
> 2) A Client must use the createPassiveKeepAliveFilter?
>
>
> On 10/16/07, mat <fo...@gmail.com> wrote:
> >
> > +1 very important (Definitely a MUST filter)
> >
> > On 9/27/07, Mark <el...@gmail.com> wrote:
> > >
> > > [X]: +1, import
> > >
> > > On 9/26/07, Michael Grundvig < mike@electrotank.com > wrote:
> > > >
> > > > +1, import
> > > >
> > > > ----- Original Message -----
> > > > From: "Trustin Lee" < trustin@gmail.com>
> > > > To: < dev@mina.apache.org>
> > > > Sent: Wednesday, September 26, 2007 5:24 PM
> > > > Subject: [VOTE] Import KeepAlive functionality contribution under
> > > MINA PMC
> > > >
> > > >
> > > > > Hi again.
> > > > >
> > > > > There's an important contribution from Eero Nevalainen that will
> > > help
> > > > > users to implement sending a protocol-specific keep-alive message
> > > > > periodically:
> > > > >
> > > > > https://issues.apache.org/jira/browse/DIRMINA-354
> > > > >
> > > > > It's just a few classes, so I thought we could just import the
> > > code,
> > > > > but procedure is procedure. :)
> > > > >
> > > > > Software grant has been received long time ago, and also again, it
> > > was
> > > > > not processed on time due to my laziness. There's no reason not
> > > to
> > > > > import these classes IMO; it's just a few classes that makes
> > > people's
> > > > > life a lot easier.
> > > > >
> > > > > Once vote passes, I will start the paperwork and all the required
> > > > > procedure.
> > > > >
> > > > > [ ]: +1, import
> > > > > [ ] 0, abstain
> > > > > [ ]: -1, don't import
> > > > >
> > > > > Cheers,
> > > > > Trustin
> > > > > --
> > > > > what we call human nature is actually human habit
> > > > > --
> > > > > http://gleamynode.net/
> > > > > --
> > > > > PGP Key ID: 0x0255ECA6
> > > >
> > > >
> > >
> > >
> > > --
> > > ..Cheers
> > > Mark
> > >
> >
> >
>
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by mat <fo...@gmail.com>.
Several questions on your KeepAliveFilter.
1) Why createActiveKeepAliveFilter, createPassiveKeepAliveFilter NOT return
IoFilter(A factory method is better to return the interface instead of
concrete class)? And my server or client has to extends ChainedIoHandler,
otherwises, i can't use "config.getFilterChain().addFirst" in my original
code.
2) A Client must use the createPassiveKeepAliveFilter?
On 10/16/07, mat <fo...@gmail.com> wrote:
>
> +1 very important (Definitely a MUST filter)
>
> On 9/27/07, Mark <el...@gmail.com> wrote:
> >
> > [X]: +1, import
> >
> > On 9/26/07, Michael Grundvig <mike@electrotank.com > wrote:
> > >
> > > +1, import
> > >
> > > ----- Original Message -----
> > > From: "Trustin Lee" <tr...@gmail.com>
> > > To: < dev@mina.apache.org>
> > > Sent: Wednesday, September 26, 2007 5:24 PM
> > > Subject: [VOTE] Import KeepAlive functionality contribution under MINA
> > PMC
> > >
> > >
> > > > Hi again.
> > > >
> > > > There's an important contribution from Eero Nevalainen that will
> > help
> > > > users to implement sending a protocol-specific keep-alive message
> > > > periodically:
> > > >
> > > > https://issues.apache.org/jira/browse/DIRMINA-354
> > > >
> > > > It's just a few classes, so I thought we could just import the code,
> > > > but procedure is procedure. :)
> > > >
> > > > Software grant has been received long time ago, and also again, it
> > was
> > > > not processed on time due to my laziness. There's no reason not to
> > > > import these classes IMO; it's just a few classes that makes
> > people's
> > > > life a lot easier.
> > > >
> > > > Once vote passes, I will start the paperwork and all the required
> > > > procedure.
> > > >
> > > > [ ]: +1, import
> > > > [ ] 0, abstain
> > > > [ ]: -1, don't import
> > > >
> > > > Cheers,
> > > > Trustin
> > > > --
> > > > what we call human nature is actually human habit
> > > > --
> > > > http://gleamynode.net/
> > > > --
> > > > PGP Key ID: 0x0255ECA6
> > >
> > >
> >
> >
> > --
> > ..Cheers
> > Mark
> >
>
>
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by mat <fo...@gmail.com>.
+1 very important (Definitely a MUST filter)
On 9/27/07, Mark <el...@gmail.com> wrote:
>
> [X]: +1, import
>
> On 9/26/07, Michael Grundvig <mi...@electrotank.com> wrote:
> >
> > +1, import
> >
> > ----- Original Message -----
> > From: "Trustin Lee" <tr...@gmail.com>
> > To: <de...@mina.apache.org>
> > Sent: Wednesday, September 26, 2007 5:24 PM
> > Subject: [VOTE] Import KeepAlive functionality contribution under MINA
> PMC
> >
> >
> > > Hi again.
> > >
> > > There's an important contribution from Eero Nevalainen that will help
> > > users to implement sending a protocol-specific keep-alive message
> > > periodically:
> > >
> > > https://issues.apache.org/jira/browse/DIRMINA-354
> > >
> > > It's just a few classes, so I thought we could just import the code,
> > > but procedure is procedure. :)
> > >
> > > Software grant has been received long time ago, and also again, it was
> > > not processed on time due to my laziness. There's no reason not to
> > > import these classes IMO; it's just a few classes that makes people's
> > > life a lot easier.
> > >
> > > Once vote passes, I will start the paperwork and all the required
> > > procedure.
> > >
> > > [ ]: +1, import
> > > [ ] 0, abstain
> > > [ ]: -1, don't import
> > >
> > > Cheers,
> > > Trustin
> > > --
> > > what we call human nature is actually human habit
> > > --
> > > http://gleamynode.net/
> > > --
> > > PGP Key ID: 0x0255ECA6
> >
> >
>
>
> --
> ..Cheers
> Mark
>
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Mark <el...@gmail.com>.
[X]: +1, import
On 9/26/07, Michael Grundvig <mi...@electrotank.com> wrote:
>
> +1, import
>
> ----- Original Message -----
> From: "Trustin Lee" <tr...@gmail.com>
> To: <de...@mina.apache.org>
> Sent: Wednesday, September 26, 2007 5:24 PM
> Subject: [VOTE] Import KeepAlive functionality contribution under MINA PMC
>
>
> > Hi again.
> >
> > There's an important contribution from Eero Nevalainen that will help
> > users to implement sending a protocol-specific keep-alive message
> > periodically:
> >
> > https://issues.apache.org/jira/browse/DIRMINA-354
> >
> > It's just a few classes, so I thought we could just import the code,
> > but procedure is procedure. :)
> >
> > Software grant has been received long time ago, and also again, it was
> > not processed on time due to my laziness. There's no reason not to
> > import these classes IMO; it's just a few classes that makes people's
> > life a lot easier.
> >
> > Once vote passes, I will start the paperwork and all the required
> > procedure.
> >
> > [ ]: +1, import
> > [ ] 0, abstain
> > [ ]: -1, don't import
> >
> > Cheers,
> > Trustin
> > --
> > what we call human nature is actually human habit
> > --
> > http://gleamynode.net/
> > --
> > PGP Key ID: 0x0255ECA6
>
>
--
..Cheers
Mark
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Michael Grundvig <mi...@electrotank.com>.
+1, import
----- Original Message -----
From: "Trustin Lee" <tr...@gmail.com>
To: <de...@mina.apache.org>
Sent: Wednesday, September 26, 2007 5:24 PM
Subject: [VOTE] Import KeepAlive functionality contribution under MINA PMC
> Hi again.
>
> There's an important contribution from Eero Nevalainen that will help
> users to implement sending a protocol-specific keep-alive message
> periodically:
>
> https://issues.apache.org/jira/browse/DIRMINA-354
>
> It's just a few classes, so I thought we could just import the code,
> but procedure is procedure. :)
>
> Software grant has been received long time ago, and also again, it was
> not processed on time due to my laziness. There's no reason not to
> import these classes IMO; it's just a few classes that makes people's
> life a lot easier.
>
> Once vote passes, I will start the paperwork and all the required
> procedure.
>
> [ ]: +1, import
> [ ] 0, abstain
> [ ]: -1, don't import
>
> Cheers,
> Trustin
> --
> what we call human nature is actually human habit
> --
> http://gleamynode.net/
> --
> PGP Key ID: 0x0255ECA6
Re: [VOTE] Import KeepAlive functionality contribution under MINA
PMC
Posted by Eero Nevalainen <ee...@indagon.com>.
+1 import. The code is actually a bit ugly to use, since I have to store
and retrieve the IoSession specific values as session attributes and the
sessionCreated is called elsewhere... In MINA 2.0 this function can be
properly implemented as an IoFilter, since 2.0 allows to have
ProtocolCodecFilters earlier in the chain. But import first, refactor
later..
-Eero
Julien Vermillard wrote:
> +1 import, the code is looking nice, and it's a very common need.
> Julien
>
> On Thu, 27 Sep 2007 07:24:45 +0900
> "Trustin Lee" <tr...@gmail.com> wrote:
>
>> Hi again.
>>
>> There's an important contribution from Eero Nevalainen that will help
>> users to implement sending a protocol-specific keep-alive message
>> periodically:
>>
>> https://issues.apache.org/jira/browse/DIRMINA-354
>>
>> It's just a few classes, so I thought we could just import the code,
>> but procedure is procedure. :)
>>
>> Software grant has been received long time ago, and also again, it was
>> not processed on time due to my laziness. There's no reason not to
>> import these classes IMO; it's just a few classes that makes people's
>> life a lot easier.
>>
>> Once vote passes, I will start the paperwork and all the required
>> procedure.
>>
>> [ ]: +1, import
>> [ ] 0, abstain
>> [ ]: -1, don't import
>>
>> Cheers,
>> Trustin
>
Re: [VOTE] Import KeepAlive functionality contribution under MINA
PMC
Posted by Julien Vermillard <jv...@archean.fr>.
+1 import, the code is looking nice, and it's a very common need.
Julien
On Thu, 27 Sep 2007 07:24:45 +0900
"Trustin Lee" <tr...@gmail.com> wrote:
> Hi again.
>
> There's an important contribution from Eero Nevalainen that will help
> users to implement sending a protocol-specific keep-alive message
> periodically:
>
> https://issues.apache.org/jira/browse/DIRMINA-354
>
> It's just a few classes, so I thought we could just import the code,
> but procedure is procedure. :)
>
> Software grant has been received long time ago, and also again, it was
> not processed on time due to my laziness. There's no reason not to
> import these classes IMO; it's just a few classes that makes people's
> life a lot easier.
>
> Once vote passes, I will start the paperwork and all the required
> procedure.
>
> [ ]: +1, import
> [ ] 0, abstain
> [ ]: -1, don't import
>
> Cheers,
> Trustin
Re: [VOTE] Import KeepAlive functionality contribution under MINA PMC
Posted by Enrique Rodriguez <en...@gmail.com>.
On 9/26/07, Trustin Lee <tr...@gmail.com> wrote:
> ...
[ X ]: +1, import
[ ] 0, abstain
[ ]: -1, don't import
Enrique