revert

Author: shacharPash

src/NRedisStack/Graph/DataTypes/Edge.cs

@@ -26,6 +26,7 @@ public class Edge : GraphEntity
         /// <value></value>
         public long Destination { get; set; }
 
+        // TODO: check if this is needed:
         /// <summary>
         /// Overriden from the base `Equals` implementation. In addition to the expected behavior of checking
         /// reference equality, we'll also fall back and check to see if the: Source, Destination, and RelationshipType

src/NRedisStack/Graph/DataTypes/GraphEntity.cs

@@ -12,6 +12,7 @@ public abstract class GraphEntity
 
         public IDictionary<string, object> PropertyMap = new Dictionary<string, object>();
 
+        // TODO: check if this is needed:
         /// <summary>
         /// Overriden Equals that considers the equality of the entity ID as well as the equality of the
         /// properties that each entity has.

src/NRedisStack/Graph/DataTypes/Node.cs

@@ -16,6 +16,7 @@ public Node()
             Labels = new List<string>();
         }
 
+        // TODO: check if this is needed:
         /// <summary>
         /// Overriden member that checks to see if the names of the labels of a node are equal
         /// (in addition to base `Equals` functionality).

src/NRedisStack/Graph/DataTypes/Path.cs

@@ -20,12 +20,7 @@ public Path(IList<Node> nodes, IList<Edge> edges)
             Edges = new ReadOnlyCollection<Edge>(edges);
         }
 
-
-        /// <summary>
-        /// How many edges exist on this path.
-        /// </summary>
-        public int Length => Edges.Count;
-
+        // TODO: check if this is needed:
         /// <summary>
         /// Overriden `Equals` method that will consider the equality of the Nodes and Edges between two paths.
         /// </summary>
@@ -73,6 +68,7 @@ public override int GetHashCode()
             }
         }
 
+        // TODO: check if this is needed:
         /// <summary>
         /// Overridden `ToString` method that will emit a string based on the string values of the nodes and edges
         /// on the path.

src/NRedisStack/Graph/GraphCommandBuilder.cs

@@ -8,14 +8,6 @@ public static class GraphCommandBuilder
     {
         internal static readonly object CompactQueryFlag = "--COMPACT";
 
-        /// <inheritdoc/>
-        public static SerializedCommand Query(string graphName, string query, IDictionary<string, object> parameters, long? timeout = null)
-        {
-            var preparedQuery = PrepareQuery(query, parameters);
-
-            return Query(graphName, preparedQuery, timeout);
-        }
-
         /// <inheritdoc/>
         public static SerializedCommand Query(string graphName, string query, long? timeout = null)
         {
@@ -25,14 +17,6 @@ public static SerializedCommand Query(string graphName, string query, long? time
             return new SerializedCommand(GRAPH.QUERY, args);
         }
 
-        /// <inheritdoc/>
-        public static SerializedCommand RO_Query(string graphName, string query, IDictionary<string, object> parameters, long? timeout = null)
-        {
-            var preparedQuery = PrepareQuery(query, parameters);
-
-            return RO_Query(graphName, preparedQuery, timeout);
-        }
-
         /// <inheritdoc/>
         public static SerializedCommand RO_Query(string graphName, string query, long? timeout = null)
         {

src/NRedisStack/Graph/Header.cs

@@ -41,6 +41,7 @@ internal Header(RedisResult result)
             }
         }
 
+        // TODO: check if this is needed:
         public override bool Equals(object? obj)
         {
             if (obj == null) return this == null;

src/NRedisStack/Graph/Point.cs

@@ -23,6 +23,7 @@ public Point(List<double> values)
             this.longitude = values[1];
         }
 
+        // TODO: check if this is needed:
         public override bool Equals(object? obj)
         {
             if (obj == null) return this == null;

src/NRedisStack/Graph/Record.cs

@@ -58,6 +58,7 @@ internal Record(List<string> header, List<object> values)
         /// </summary>
         public int Size => Header.Count;
 
+        // TODO: check if this is needed:
         public override bool Equals(object? obj)
         {
             if (obj == null) return this == null;

tests/NRedisStack.Tests/Search/SearchTests.cs

@@ -672,7 +672,7 @@ public void AlterAdd()
         Assert.Equal(100, info.NumDocs);
         Assert.Equal("300", info.MaxDocId);
         Assert.Equal(102, info.NumTerms);
-        Assert.Equal(800, info.NumRecords);
+        Assert.Equal(800, info.NumRecords); // TODO: why is this 800? 
         Assert.Equal(0.004291534423828125, info.InvertedSzMebibytes);
         Assert.Equal(0, info.VectorIndexSzMebibytes);
         Assert.Equal(208, info.TotalInvertedIndexBlocks);
@@ -691,7 +691,6 @@ public void AlterAdd()
         Assert.Equal(4, info.NumberOfUses);
         Assert.Equal(7, info.GcStats.Count);
         Assert.Equal(4, info.CursorStats.Count);
-
     }
 
     [Fact]