[ https://issues.apache.org/jira/browse/SOLR-3913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17876974#comment-17876974 ]
ASF subversion and git services commented on SOLR-3913: ------------------------------------------------------- Commit c316c7fab249d45443dbd1c1ee0c6a850c999dc9 in solr's branch refs/heads/branch_9x from Eric Pugh [ https://gitbox.apache.org/repos/asf?p=solr.git;h=c316c7fab24 ] SOLR-3913: Optimize the commit/optimize cycle in the PostTool (#2667) Arguably you shouldn't be able to pass in the skip-commit AND the optimize options as the same time...? (cherry picked from commit 94146c1c3084d18038a95c51b0e90e605b187922) > SimplePostTool optimize does a redundant commit > ----------------------------------------------- > > Key: SOLR-3913 > URL: https://issues.apache.org/jira/browse/SOLR-3913 > Project: Solr > Issue Type: Improvement > Components: scripts and tools > Reporter: David Smiley > Assignee: Eric Pugh > Priority: Minor > Labels: pull-request-available > Fix For: 9.8 > > Time Spent: 20m > Remaining Estimate: 0h > > At the end of SimplePostTool.execute() there is: > {code} > if (commit) commit(); > if (optimize) optimize(); > {code} > Each of these calls involves a separate request to Solr. The thing is, an > optimize internally commits, and so the logic should forgo committing is > optimize is true. > And as an aside, I think the 1kb pipe() buffer on line 893 is too small; it > should be around 8kb (8192) bytes which is the same value as > BufferedInputStream's default. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org