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 Norman Maurer <nm...@byteaction.de> on 2006/09/05 21:19:17 UTC
Re: [jira] Updated: (JAMES-590) Add commands to RemoteManager to
corpusfeed JDBCBayesianAnalyzer
Hi Bernd,
thx for the patch. I change some stuff and commited it (Hope that is
ok). Here is what i did:
- Remove not needed imports
- Fix some spelling
- Add more javadocs
- Change RemoteManagerHandler to not disconnect if an
BayesianAnalyzerManagementException is caught
I test the corpusfeeding via dir and mbox. The JMX was not tested by me.
I think that was allready done by you when developing.
I think we can also now support this commands by default cause we now
have derby support for BayesianAnalyzer.
What the other guys think ?
bye
Norman
Am Dienstag, den 05.09.2006, 13:24 +0200 schrieb Bernd Fondermann:
> There are some reasons for not committing this directly:
> o Norman recently developed and added the functionality, he tested it
> and it was all good.
> o Now, I did some more than minor refactorings including changing the
> component API and thus the integration of the components.
> o Currently, I am not able to test the Bayesian management, including
> my new code, by myself. (time and workplace restrictions)
>
> I think it would be unfair to unintentionally break the working code
> from Norman for the pure reason of (subjective) beauty. Committing
> untested code is not acceptable.
>
> So I chose to put it in the JIRA until myself, Norman or others have
> time to pick it up.
>
> I only tested the availability of the new BayesianAnalyzerManagement
> JMX component and that part seems to works fine.
>
> Should've added these notes to the patch in the first place...
>
> > So feel free to just commit it using the "JAMES-590" identifier in the
> > log so that it will automatically appear in the JIRA issue.
>
> Are you saying, JIRA parses commit logs and links them to issues?
>
> Bernd
>
> On 9/5/06, Stefano Bagnara <ap...@bago.org> wrote:
> > Bernd,
> >
> > Imho you should directly commit this kind of patches. You are a
> > committer (a PMC one) and we use the Commit-Then-Review approach with
> > our trunk.
> >
> > So feel free to just commit it using the "JAMES-590" identifier in the
> > log so that it will automatically appear in the JIRA issue.
> >
> > This make it easier to review either the diff and the final result
> > without having to apply patches manually.
> >
> > This is not a critic, I just want to let you know that I trust you and I
> > think you produce good quality code, so feel free to commit to trunk (we
> > can always revert it later if anyone has problem with the code)
> >
> > Stefano
> >
> > Bernd Fondermann (JIRA) wrote:
> > > [ http://issues.apache.org/jira/browse/JAMES-590?page=all ]
> > >
> > > Bernd Fondermann updated JAMES-590:
> > > -----------------------------------
> > >
> > > Attachment: bayesian_jmx.patch
> > >
> > > reduced the number of thrown excpetions to one: BayesianManagementException. (this is best practice. components should not put the burden of handling internal errors on the caller)
> > >
> > > made management functionality available for JMX.
> > >
> > >> Add commands to RemoteManager to corpusfeed JDBCBayesianAnalyzer
> > >> ----------------------------------------------------------------
> > >>
> > >> Key: JAMES-590
> > >> URL: http://issues.apache.org/jira/browse/JAMES-590
> > >> Project: James
> > >> Issue Type: New Feature
> > >> Reporter: Norman Maurer
> > >> Assigned To: Norman Maurer
> > >> Priority: Minor
> > >> Fix For: 3.0
> > >>
> > >> Attachments: bayesian_jmx.patch
> > >>
> > >>
> > >> We should add commands to RemoteManager to allow an admin to corpus feed JDBCBayesianAnalyzer. This whould allow an new user to train the spamfilter with an corpus of ham or spam without sending each spam or ham with mail
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> > For additional commands, e-mail: server-dev-help@james.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
> !EXCUBATOR:1,44fd5e7645111262318898!
Re: [jira] Updated: (JAMES-590) Add commands to RemoteManager to corpusfeed JDBCBayesianAnalyzer
Posted by Bernd Fondermann <be...@googlemail.com>.
On 9/5/06, Norman Maurer <nm...@byteaction.de> wrote:
> Hi Bernd,
>
> thx for the patch. I change some stuff and commited it (Hope that is
> ok). Here is what i did:
>
> - Remove not needed imports
> - Fix some spelling
> - Add more javadocs
> - Change RemoteManagerHandler to not disconnect if an
> BayesianAnalyzerManagementException is caught
>
> I test the corpusfeeding via dir and mbox. The JMX was not tested by me.
> I think that was allready done by you when developing.
thanks, for doing this for me. I like the method name change.
I will retest the JMX part.
So I guess we can finally close the JIRA...
Thanks again,
Bernd
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org