[ https://issues.apache.org/jira/browse/SOLR-17758?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17952632#comment-17952632 ]
ASF subversion and git services commented on SOLR-17758: -------------------------------------------------------- Commit d8e05ac476299ee7cd5d726ff95162ce4dd3d1b0 in solr's branch refs/heads/branch_9x from Jason Gerlowski [ https://gitbox.apache.org/repos/asf?p=solr.git;h=d8e05ac4762 ] SOLR-17758: Fix limiting-URP "warnOnly" mode (#3348) Due to a bug in the initial implementation, the "warnOnly" mode of the NumFieldLimiting "URP" never worked as designed. The "warn" codepath failed to call `processAdd` for the next URP in the chain. This makes the "warn only" behavior indistinguishable from the "hard limit" behavior from a user perspective. This commit fixes this bug. It also adds integration testing for several "warn only" mode related scenarios, to prevent against future regressions. > NumFieldLimiting URP "warnOnly" mode broken > ------------------------------------------- > > Key: SOLR-17758 > URL: https://issues.apache.org/jira/browse/SOLR-17758 > Project: Solr > Issue Type: Bug > Components: UpdateRequestProcessors > Affects Versions: 9.8.1 > Reporter: Jason Gerlowski > Priority: Minor > Labels: pull-request-available > Time Spent: 40m > Remaining Estimate: 0h > > NumFieldLimitingUpdateProcessorFactory (introduced in SOLR-17192) aims to > offer a "warnOnly" mode that logs a warning when the maximum number of fields > is exceeded. > But the "warnOnly" code path doesn't trigger any subsequent processors in the > chain. So in effect, both modes will prevent new documents from being added > once the limit has been exceeded. > We should rework this logic so that the warnOnly=true codepath allows > documents to be indexed as expected. -- 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