You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2020/08/22 18:42:28 UTC

[lucene-solr] 09/22: @589 Fix NPE.

This is an automated email from the ASF dual-hosted git repository.

markrmiller pushed a commit to branch reference_impl
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git

commit eaee64eac96cf997c549da9a914ef5a5f91ea390
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Thu Aug 20 15:22:18 2020 -0500

    @589 Fix NPE.
---
 .../java/org/apache/solr/ltr/SolrQueryRequestContextUtils.java    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/solr/contrib/ltr/src/java/org/apache/solr/ltr/SolrQueryRequestContextUtils.java b/solr/contrib/ltr/src/java/org/apache/solr/ltr/SolrQueryRequestContextUtils.java
index 2cff28f..fb24411 100644
--- a/solr/contrib/ltr/src/java/org/apache/solr/ltr/SolrQueryRequestContextUtils.java
+++ b/solr/contrib/ltr/src/java/org/apache/solr/ltr/SolrQueryRequestContextUtils.java
@@ -38,7 +38,9 @@ public class SolrQueryRequestContextUtils {
   /** feature logger accessors **/
 
   public static void setFeatureLogger(SolrQueryRequest req, FeatureLogger featureLogger) {
-    req.getContext().put(FEATURE_LOGGER, featureLogger);
+    if (featureLogger != null) {
+      req.getContext().put(FEATURE_LOGGER, featureLogger);
+    }
   }
 
   public static FeatureLogger getFeatureLogger(SolrQueryRequest req) {
@@ -72,7 +74,9 @@ public class SolrQueryRequestContextUtils {
   /** feature vector store name accessors **/
 
   public static void setFvStoreName(SolrQueryRequest req, String fvStoreName) {
-    req.getContext().put(STORE, fvStoreName);
+    if (fvStoreName != null) {
+      req.getContext().put(STORE, fvStoreName);
+    }
   }
 
   public static String getFvStoreName(SolrQueryRequest req) {