Cleanup duplicate FilterScorable in QueryPhaseCollector (elastic#112603)

It's in the title, we can dedup the code here.

server/src/main/java/org/elasticsearch/search/query/QueryPhaseCollector.java

@@ -189,14 +189,10 @@ public LeafCollector getLeafCollector(LeafReaderContext context) throws IOExcept
             return new FilterLeafCollector(topDocsLeafCollector) {
                 @Override
                 public void setScorer(Scorable scorer) throws IOException {
-                    super.setScorer(new FilterScorable(scorer) {
-                        @Override
-                        public void setMinCompetitiveScore(float minScore) {
-                            // Ignore calls to setMinCompetitiveScore. The top docs collector may try to skip low
-                            // scoring hits, but the overall score_mode won't allow it because an aggs collector
-                            // was originally provided which never supports TOP_SCORES is not supported for aggs
-                        }
-                    });
+                    // Ignore calls to setMinCompetitiveScore. The top docs collector may try to skip low
+                    // scoring hits, but the overall score_mode won't allow it because an aggs collector
+                    // was originally provided which never supports TOP_SCORES is not supported for aggs
+                    super.setScorer(wrapToIgnoreMinCompetitiveScore(scorer));
                 }
 
                 @Override
@@ -208,6 +204,13 @@ public DocIdSetIterator competitiveIterator() throws IOException {
         return new CompositeLeafCollector(postFilterBits, topDocsLeafCollector, aggsLeafCollector);
     }
 
+    private static FilterScorable wrapToIgnoreMinCompetitiveScore(Scorable scorer) {
+        return new FilterScorable(scorer) {
+            @Override
+            public void setMinCompetitiveScore(float minScore) {}
+        };
+    }
+
     private class TopDocsLeafCollector implements LeafCollector {
         private final Bits postFilterBits;
         private final LeafCollector topDocsLeafCollector;
@@ -262,14 +265,10 @@ public void setScorer(Scorable scorer) throws IOException {
             if (cacheScores && topDocsLeafCollector != null && aggsLeafCollector != null) {
                 scorer = ScoreCachingWrappingScorer.wrap(scorer);
             }
-            scorer = new FilterScorable(scorer) {
-                @Override
-                public void setMinCompetitiveScore(float minScore) {
-                    // Ignore calls to setMinCompetitiveScore so that if the top docs collector
-                    // wants to skip low-scoring hits, the aggs collector still sees all hits.
-                    // this is important also for terminate_after in case used when total hits tracking is early terminated.
-                }
-            };
+            // Ignore calls to setMinCompetitiveScore so that if the top docs collector
+            // wants to skip low-scoring hits, the aggs collector still sees all hits.
+            // this is important also for terminate_after in case used when total hits tracking is early terminated.
+            scorer = wrapToIgnoreMinCompetitiveScore(scorer);
             if (topDocsLeafCollector != null) {
                 topDocsLeafCollector.setScorer(scorer);
             }