[
https://issues.apache.org/jira/browse/SOLR-7659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14580669#comment-14580669
]
Shawn Heisey edited comment on SOLR-7659 at 6/10/15 3:46 PM:
-------------------------------------------------------------
It's not completely necessary to rename the method, but it seemed like a good
thing to do based on changing what the code does. Within the Solr source, it
is only used locally, and the number of users that are likely to write code
extending that particular inner class should be very low.
was (Author: elyograg):
It's not completely necessary to rename the method, but it seemed like a good
thing to do based on changing what the code does. Within the Solr source, it
is only used locally, and the number of users that are likely to write code
extending the class should be very low.
> DirectoryFileStream.releaseCommitPointAndExtendReserve -- should it extend
> first?
> ---------------------------------------------------------------------------------
>
> Key: SOLR-7659
> URL: https://issues.apache.org/jira/browse/SOLR-7659
> Project: Solr
> Issue Type: Improvement
> Components: replication (java)
> Affects Versions: 5.2.1
> Reporter: Shawn Heisey
> Priority: Minor
> Fix For: 5.3, Trunk
>
> Attachments: SOLR-7659.patch
>
>
> After a misunderstanding on my part, leading to an incorrect documentation
> edit, [~shalinmangar] helped me understand how the commit reserve duration
> works.
> The resulting discussion on the dev list brought up a possible problem in
> ReplicationHandler. We wondered whether it might be possible for another
> thread to sneak in a commit point deletion in between the two statements
> found in the releaseCommitPointAndExtendReserve method on the
> DirectoryFileStream object. If it is safe to do so, I propose reversing
> those two statements so the reserve extension is done before releasing the
> commit point.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
