[
https://issues.apache.org/jira/browse/LUCENE-5569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14293270#comment-14293270
]
Adrien Grand commented on LUCENE-5569:
--------------------------------------
Actually I think this 5.0 break is one of the easiest ones to fix when
upgrading. Although it might generate lots of compilation errors, especially if
you use experimental APIs like oal.search.Collector, it's very easy to fix
using eg. sed to replace all occurrences of AtomicReader with LeafReader (which
will also rename AtomicReaderContext to LeafReaderContext).
> Rename AtomicReader to LeafReader
> ---------------------------------
>
> Key: LUCENE-5569
> URL: https://issues.apache.org/jira/browse/LUCENE-5569
> Project: Lucene - Core
> Issue Type: Improvement
> Reporter: Adrien Grand
> Assignee: Ryan Ernst
> Priority: Blocker
> Fix For: 5.0
>
> Attachments: LUCENE-5569.patch, LUCENE-5569.patch
>
>
> See LUCENE-5527 for more context: several of us seem to prefer {{Leaf}} to
> {{Atomic}}.
> Talking from my experience, I was a bit confused in the beginning that this
> thing is named {{AtomicReader}}, since {{Atomic}} is otherwise used in Java
> in the context of concurrency. So maybe renaming it to {{Leaf}} would help
> remove this confusion and also carry the information that these readers are
> used as leaves of top-level readers?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
