james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrzej Rusin (JIRA)" <server-...@james.apache.org>
Subject [jira] [Commented] (JAMES-1441) Un-staticize the org.apache.james.mailbox.store.search.MessageSearches
Date Tue, 04 Dec 2012 14:06:58 GMT

    [ https://issues.apache.org/jira/browse/JAMES-1441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13509748#comment-13509748
] 

Andrzej Rusin commented on JAMES-1441:
--------------------------------------

Eric,
I am no longer planning to subclass it, that's why I left the methods as private.
My original intention was to override the full text search methods with searching a custom
SOLR repository, and it's probably the only place where it makes sense to subclass this implementation.

As for synchronization, removed. Note that each call of iterator() will re-search; hope it's
the correct behavior.

As for isMatch methods, one of them needs to be public so the tests can access it.


Regards,
Andrzej
                
> Un-staticize the org.apache.james.mailbox.store.search.MessageSearches
> ----------------------------------------------------------------------
>
>                 Key: JAMES-1441
>                 URL: https://issues.apache.org/jira/browse/JAMES-1441
>             Project: James Server
>          Issue Type: Improvement
>          Components: MailStore & MailRepository
>    Affects Versions: Trunk
>         Environment: all
>            Reporter: Andrzej Rusin
>            Priority: Trivial
>         Attachments: JAMES-1441.patch, JAMES-1441-v2.patch
>
>
> To allow some overriding, and also make it more elegant, the following would be nice:
> A convert the static methods to non-static,
> B convert certain methods from private to protected,
> C remove the main processing from the constructor (necessary for A, and also a good practice)

--
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-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message