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

Shalin Shekhar Mangar resolved SOLR-6224.
-----------------------------------------
       Resolution: Fixed
    Fix Version/s:     (was: 4.10)
                   Trunk
                   5.0
         Assignee: Shalin Shekhar Mangar

> Post soft-commit callbacks are called before soft commit actually happens
> -------------------------------------------------------------------------
>
>                 Key: SOLR-6224
>                 URL: https://issues.apache.org/jira/browse/SOLR-6224
>             Project: Solr
>          Issue Type: Bug
>          Components: update
>    Affects Versions: 4.9
>            Reporter: Shalin Shekhar Mangar
>            Assignee: Shalin Shekhar Mangar
>            Priority: Minor
>             Fix For: 5.0, Trunk
>
>         Attachments: SOLR-6224.patch, SOLR-6224.patch
>
>
> DirectUpdateHandler2 has the following code:
> {code}
> if (!cmd.softCommit) {
>           synchronized (solrCoreState.getUpdateLock()) { // sync is currently 
> needed to prevent preCommit
>                                 // from being called between preSoft and
>                                 // postSoft... see postSoft comments.
>             if (ulog != null) ulog.preCommit(cmd);
>           }
>           
>           // SolrCore.verbose("writer.commit() start writer=",writer);
>           if (writer.hasUncommittedChanges()) {
>             final Map<String,String> commitData = new HashMap<>();
>             commitData.put(SolrIndexWriter.COMMIT_TIME_MSEC_KEY,
>                 String.valueOf(System.currentTimeMillis()));
>             writer.setCommitData(commitData);
>             writer.commit();
>           } else {
>             log.info("No uncommitted changes. Skipping IW.commit.");
>           }
>           // SolrCore.verbose("writer.commit() end");
>           numDocsPending.set(0);
>           callPostCommitCallbacks();
>         } else {
>           callPostSoftCommitCallbacks();
>         }
> {code}
> As evident in the code above, if cmd.softCommit is true then the post soft 
> commit hooks are called first and the actual soft commit happens in a block 
> of code after it.



--
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