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

Yonik Seeley commented on LUCENE-5569:
--------------------------------------

FWIW, I agree with Tim.  There's not a right or wrong to it... it's a judgement 
call.  It's clear to me that the bar these days to renaming public APIs is far 
too low... but I appear to be outnumbered.

> 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
>            Priority: Minor
>             Fix For: 5.0
>
>         Attachments: 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.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to