[ https://issues.apache.org/jira/browse/SOLR-13474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16844158#comment-16844158 ]
ASF subversion and git services commented on SOLR-13474: -------------------------------------------------------- Commit 4dd3186c0f1d7ab78f30981acda527b85cfb8bdc in lucene-solr's branch refs/heads/branch_8x from Chris M. Hostetter [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=4dd3186 ] SOLR-13474: Fix "Search is temporarily disabled" logic to be consistent for entire request SOLR-13469: Fix rejected requests during full disk full + IndexFetch to use 503 not 403 (cherry picked from commit 6cc67e433e191b5c0450a4d2fc06de26939ac920) Conflicts: solr/CHANGES.txt > Fix "Search is temporarily disabled" logic to be consistent for entire request > ------------------------------------------------------------------------------ > > Key: SOLR-13474 > URL: https://issues.apache.org/jira/browse/SOLR-13474 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Reporter: Hoss Man > Assignee: Hoss Man > Priority: Major > Attachments: SOLR-13474.patch, SOLR-13474.patch > > > Where/How the "Search is temporarily disabled" logic was added in SOLR-12999 > can result in some weird failure cases (examples: SOLR-13470 & SOLR-13471) > due to the fact that (as currently implemented) some portions of processing > a SolrQueryRequest may see a valid searcher, but later other code paths will > get an error when trying to use the same searcher. > I think it would make more sense to move the "searchEnabled" logic out of > {{SolrQueryRequestbase.getSearcher()}} and into {{SolrCore.getSearcher()}} > (removing the need for {{SolrCore.isSearchEnabled()}} ). > This will: > * restore the long standing (and widly assumed by code) behavior of > {{SolrQueryRequestbase.getSearcher()}} that it will always return a > consistent result > * ensure that any custom plugins trying to call {{SolrCore.getSearcher()}} > directly will also get the benefits of the safety checks introduced in > SOLR-12999 -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org