[
https://issues.apache.org/jira/browse/SOLR-17504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17921869#comment-17921869
]
ASF subversion and git services commented on SOLR-17504:
--------------------------------------------------------
Commit d48869a77d626e4546de6e8fe9c7f4251fff5ea8 in solr's branch
refs/heads/branch_9x from Houston Putman
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=d48869a77d6 ]
SOLR-17504: Ensure DirectUpdateHandlerWithUpdateLogTest always uses updateLog
(cherry picked from commit 3be16e25517f46b9a4cdfd35bd0036d93d728743)
> CoreContainer calls UpdateHandler.commit
> ----------------------------------------
>
> Key: SOLR-17504
> URL: https://issues.apache.org/jira/browse/SOLR-17504
> Project: Solr
> Issue Type: Improvement
> Reporter: Bruno Roustant
> Assignee: Bruno Roustant
> Priority: Major
> Labels: pull-request-available
> Fix For: 9.8
>
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
> ... and DirectUpdateHandler2.closeWriter supports commit metadata.
> This proposal is about unifying calls to UpdateHandler.commit.
> CoreContainer.reload may call directly IndexWriter.commit if the core to
> reload is readonly (property of the Collection). This is an issue because it
> bypasses some logic in UpdateHandler.commit. In addition, the current code
> commits without taking the commit lock.
> DirectUpdateHandler2.closeWriter may commit if the update log is not empty
> (not committed yet). But it does not call DirectUpdateHandler2.commit(), it
> copies commit() code instead for some reasons (comments in the code mention
> test failures). The proposal is to add the same call to
> DirectUpdateHandler2.shouldCommit() as in the commit() method, otherwise
> commit metadata are lost. In addition shouldCommit() can be extended
> correctly, for example in the solr-sandbox encryption module which needs to
> intercept calls to shouldCommit().
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
