handle negative values in ranking field (#3450)

really should not have negative values, but trigger counters need to be fixed to avoid bad data in a later commit.
AudiusProject · Jul 13, 2022 · 27b598f · 27b598f
1 parent 923fed8
commit 27b598f
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/discovery-provider/src/queries/search_es.py b/discovery-provider/src/queries/search_es.py
@@ -365,16 +365,11 @@ def personalize_dsl(dsl, current_user_id, must_saved):
 def default_function_score(dsl, ranking_field):
     return {
         "query": {
-            "function_score": {
+            "script_score": {
                 "query": {"bool": dsl},
-                "functions": [
-                    {
-                        "field_value_factor": {
-                            "field": ranking_field,
-                            "modifier": "ln2p",
-                        }
-                    },
-                ],
+                "script": {
+                    "source": f" Math.log(Math.max(doc['{ranking_field}'].value, 0) + 2)"
+                },
             }
         },
     }