Test apply early limit true (#3870)

AudiusProject · Sep 14, 2022 · d46fbab · d46fbab
1 parent baed51d
commit d46fbab
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/discovery-provider/src/queries/get_trending_tracks.py b/discovery-provider/src/queries/get_trending_tracks.py
@@ -58,7 +58,7 @@ def generate_unpopulated_trending(
     # tracks we return later may be smaller than the limit.
     # If we don't limit it here, we limit it later after getting the
     # unpopulated tracks.
-    should_apply_limit_early = not exclude_premium
+    should_apply_limit_early = True  # not exclude_premium
     if should_apply_limit_early:
         sorted_track_scores = sorted_track_scores[:limit]
 
@@ -108,7 +108,7 @@ def generate_unpopulated_trending_from_mat_views(
     # tracks we return later may be smaller than the limit.
     # If we don't limit it here, we limit it later after getting the
     # unpopulated tracks.
-    should_apply_limit_early = not exclude_premium
+    should_apply_limit_early = True  # not exclude_premium
     if should_apply_limit_early:
         trending_track_ids = (
             trending_track_ids_query.order_by(

diff --git a/discovery-provider/src/queries/get_underground_trending.py b/discovery-provider/src/queries/get_underground_trending.py
@@ -202,7 +202,7 @@ def wrapped():
         # tracks we return later may be smaller than the limit.
         # If we don't limit it here, we limit it later after getting the
         # unpopulated tracks.
-        should_apply_limit_early = not SHOULD_TRENDING_EXCLUDE_PREMIUM_TRACKS
+        should_apply_limit_early = True  # not SHOULD_TRENDING_EXCLUDE_PREMIUM_TRACKS
         if should_apply_limit_early:
             sorted_tracks = sorted_tracks[:UNDERGROUND_TRENDING_LENGTH]