Revert "Add debug logs for Lucene query execution #2831"

FoundationDB · Sep 11, 2024 · 8c94566 · 8c94566
1 parent d7bee40
commit 8c94566
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 18 deletions.
diff --git a/...re/src/main/java/com/apple/foundationdb/record/query/plan/plans/RecordQueryIndexPlan.java b/...re/src/main/java/com/apple/foundationdb/record/query/plan/plans/RecordQueryIndexPlan.java
@@ -300,12 +300,6 @@ private <M extends Message> RecordCursor<QueryResult> executeUsingRemoteFetch(@N
     @Override
     public <M extends Message> RecordCursor<IndexEntry> executeEntries(@Nonnull FDBRecordStoreBase<M> store, @Nonnull EvaluationContext context,
                                                                        @Nullable byte[] continuation, @Nonnull ExecuteProperties executeProperties) {
-        if (LOGGER.isDebugEnabled()) {
-            LOGGER.debug(KeyValueLogMessage.build("executeEntries")
-                    .addKeyAndValue(LogMessageKeys.INDEX_NAME, indexName)
-                    .toString());
-        }
-
         final RecordMetaData metaData = store.getRecordMetaData();
         final Index index = metaData.getIndex(indexName);
         final IndexScanBounds scanBounds = scanParameters.bind(store, index, context);

diff --git a/...-lucene/src/main/java/com/apple/foundationdb/record/lucene/LuceneScanQueryParameters.java b/...-lucene/src/main/java/com/apple/foundationdb/record/lucene/LuceneScanQueryParameters.java
@@ -28,8 +28,6 @@
 import com.apple.foundationdb.record.PlanHashable;
 import com.apple.foundationdb.record.PlanSerializable;
 import com.apple.foundationdb.record.PlanSerializationContext;
-import com.apple.foundationdb.record.logging.KeyValueLogMessage;
-import com.apple.foundationdb.record.logging.LogMessageKeys;
 import com.apple.foundationdb.record.metadata.Index;
 import com.apple.foundationdb.record.planprotos.PIndexScanParameters;
 import com.apple.foundationdb.record.planprotos.PLuceneScanQueryParameters;
@@ -47,8 +45,6 @@
 import com.google.common.collect.ImmutableSet;
 import org.apache.lucene.search.Query;
 import org.apache.lucene.search.Sort;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -62,7 +58,6 @@
 @API(API.Status.UNSTABLE)
 public class LuceneScanQueryParameters extends LuceneScanParameters implements PlanSerializable {
     private static final ObjectPlanHash BASE_HASH = new ObjectPlanHash("Lucene-Scan-Query");
-    public static final Logger LOGGER = LoggerFactory.getLogger(LuceneScanQueryParameters.class);
 
     @Nonnull
     final LuceneQueryClause query;
@@ -141,13 +136,6 @@ public int planHash(@Nonnull PlanHashMode mode) {
     @Nonnull
     @Override
     public LuceneScanQuery bind(@Nonnull FDBRecordStoreBase<?> store, @Nonnull Index index, @Nonnull EvaluationContext context) {
-        if (LOGGER.isDebugEnabled()) {
-            LOGGER.debug(KeyValueLogMessage.build("LuceneScanQueryParameters binding")
-                    .addKeyAndValue(LogMessageKeys.INDEX_NAME, index.getName())
-                    .addKeyAndValue(LogMessageKeys.QUERY, this.query)
-                    .toString());
-        }
-
         final LuceneQueryClause.BoundQuery boundQuery = query.bind(store, index, context);
         if (luceneQueryHighlightParameters != null) {
             luceneQueryHighlightParameters.query = boundQuery.getLuceneQuery();