[ 
https://issues.apache.org/jira/browse/LUCENE-3541?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Muir updated LUCENE-3541:
--------------------------------

    Attachment: LUCENE-3541.patch

here's a prototype test (it needs to be toned down, but it does the trick), 
that fails without the patch, passes with it.

it will only fail with mmapdirectory, all the other core directories override 
this method and have some custom implementation already.
                
> remove IndexInput.copyBuf
> -------------------------
>
>                 Key: LUCENE-3541
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3541
>             Project: Lucene - Java
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Priority: Critical
>             Fix For: 3.5, 4.0
>
>         Attachments: LUCENE-3541.patch, LUCENE-3541.patch
>
>
> this looks really broken/dangerous as an instance variable.
> what happens on clone() ?! copyBytes can instead make its own array inside 
> the method.
> its protected, so ill list in the 3.x backwards breaks section since its 
> technically a backwards break.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to