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

Uwe Schindler commented on LUCENE-6129:
---------------------------------------

+1 Cool. We could extend this to other stuff in the future, too!

One small thing: {{creationThread.getId() != Thread.currentThread().getId()}} 
I think, the getId() is not really needed. Identtfy of the thread instance 
itsself is fine, too: {{creationThread != Thread.currentThread()}}
Or was there a reason to handle it like that?


> Assert that terms enums, docs enums and doc values are not leaked across 
> threads
> --------------------------------------------------------------------------------
>
>                 Key: LUCENE-6129
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6129
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Minor
>         Attachments: LUCENE-6129.patch
>
>
> Terms enums, docs enums and doc values should only be consumed in the thread 
> that they have been acquired in. We could add checks to Asserting* to make 
> sure they never leak?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to