[ https://issues.apache.org/jira/browse/SOLR-7659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14581451#comment-14581451 ]
Shawn Heisey commented on SOLR-7659: ------------------------------------ Precommit passed on branch_5x before commit. When I switched JAVA_HOME to Oracle jdk7u80, solr tests passed, but with Oracle jdk8u45, a very large number of tests were failing. > 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, 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