Revert "Improve Command parsing to consider navigation key stops when determining conditionKeys and orderKeys (#546)"

This reverts commit e29b808.

Author: jtnelson

CHANGELOG.md

@@ -13,9 +13,6 @@
 * **Dynamic Memory-Aware Eviction**: Record caches in the database now evict entires based on overall memory consumption instead of evicting after exceeding a static number of entries. Caches can grow up to an internally defined proportion of the defined `heap_size` and will be purged once this limit is exceeded or when system memory pressure necessitates garbage collection.
 * **Enhanced Diagnostic Logging**: For improved observability, DEBUG logging now emits messages whenever a cached record is evicted, allowing for more effective monitoring and troubleshooting of cache behavior.
 
-##### General Improvements and Bug Fixes
-* Fixed an issue that occurred when using a navigation key in a Criteria/Condition that was passed as a parameter to a Concourse Server command. Previously, the individual stops of navigation keys were not individually registered as *condition keys*. As a result, the `Strategy` framework didn't have all the relevant information to accurately determine all the ideal lookup sources when traversing the document graph to retrieve the values along the path. Now, in addition to the entire navigation key, each individual stop is registered as a *condition key*, which means that the `Strategy` framework will have enough information to determine if more efficient to use any index data (as opposed to table data) for lookups.
-
 #### Version 0.11.7 (April 7, 2025)
 * Fixed a bug that made it possible to leak filesystem resources by opening duplicate file descriptors for the same Segment file. At scale, this could prematurely lead to "too many open files" errors.
 * [GH-534](https://github.com/cinchapi/concourse/issues/534): Fixed a bug that caused the `CONCOURSE_HEAP_SIZE` environment variable, if set, not to be read on server startup.

concourse-server/src/main/java/com/cinchapi/concourse/server/ops/Command.java

@@ -22,7 +22,6 @@
 import java.util.Map;
 import java.util.Set;
 import java.util.stream.Collectors;
-import java.util.stream.Stream;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -41,9 +40,6 @@
 import com.cinchapi.concourse.thrift.JavaThriftBridge;
 import com.cinchapi.concourse.thrift.TCriteria;
 import com.cinchapi.concourse.thrift.TOrder;
-import com.cinchapi.concourse.validate.Keys;
-import com.cinchapi.concourse.validate.Keys.Key;
-import com.cinchapi.concourse.validate.Keys.KeyType;
 import com.google.common.base.CaseFormat;
 import com.google.common.base.MoreObjects;
 import com.google.common.base.MoreObjects.ToStringHelper;
@@ -52,7 +48,6 @@
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.Sets;
-import com.google.common.collect.Streams;
 
 /**
  * A {@link Command} describes a {@link ConcourseServer} operation and the
@@ -464,20 +459,6 @@ else if(conditionKeys == null) {
                 conditionKeys = ImmutableSet.of();
             }
 
-            // If any of the #conditionKeys are navigation keys, break them up
-            // and add each stop as a condition key
-            if(!conditionKeys.isEmpty()) {
-                conditionKeys = conditionKeys.stream().flatMap($key -> {
-                    Key key = Keys.parse($key);
-                    Stream<String> stream = Stream.of($key);
-                    if(key.type() == KeyType.NAVIGATION_KEY) {
-                        String[] stops = key.data();
-                        stream = Streams.concat(stream, Arrays.stream(stops));
-                    }
-                    return stream;
-                }).collect(Collectors.toSet());
-            }
-
             // operationTimestamp
             if(args.containsKey("time")) {
                 operationTimestamp = args.fetch("time");