PR review comments changes

Author: Niharikadutta

src/csharp/Microsoft.Spark.E2ETest/IpcTests/Sql/SparkSessionTests.cs

@@ -137,18 +137,6 @@ public void TestCreateDataFrame()
                 DataFrame df = _spark.CreateDataFrame(data);
                 ValidateDataFrame(data.Select(a => new object[] { a }), df, schema);
             }
-
-            // Calling CreateDataFrame(IEnumerable<long> _) without schema
-            {
-                var data = new List<long>(new long[] { 2L, 3L });
-                var schema = new StructType(new List<StructField>()
-                {
-                    new StructField("_1", new LongType())
-                });
-
-                DataFrame df = _spark.CreateDataFrame(data);
-                ValidateDataFrame(data.Select(a => new object[] { a }), df, schema);
-            }
         }
 
         /// <summary>
@@ -166,9 +154,7 @@ private void ValidateDataFrame(IEnumerable<object[]> data, DataFrame df, StructT
             int i = 0;
             foreach (object[] expectedValues in data)
             {
-                object[] actualValues = rows[i].Values;
-                Assert.Equal(expectedValues.Length, actualValues.Length);
-                Assert.True(actualValues.SequenceEqual(expectedValues));
+                Assert.Equal(expectedValues, rows[i].Values);
                 ++i;
             }
         }

src/csharp/Microsoft.Spark/Sql/SparkSession.cs

@@ -172,14 +172,6 @@ public DataFrame CreateDataFrame(IEnumerable<int> data) =>
         public DataFrame CreateDataFrame(IEnumerable<string> data) =>
             CreateDataFrame(ToGenericRows(data), SchemaWithSingleColumn(new StringType()));
 
-        /// <summary>
-        /// Creates a Dataframe given data as <see cref="IEnumerable"/> of type <see cref="long"/>
-        /// </summary>
-        /// <param name="data"></param>
-        /// <returns>Dataframe object</returns>
-        public DataFrame CreateDataFrame(IEnumerable<long> data) =>
-            CreateDataFrame(ToGenericRows(data), SchemaWithSingleColumn(new LongType()));
-
         /// <summary>
         /// Creates a Dataframe given data as <see cref="IEnumerable"/> of type <see cref="double"/>
         /// </summary>
@@ -196,8 +188,6 @@ public DataFrame CreateDataFrame(IEnumerable<double> data) =>
         public DataFrame CreateDataFrame(IEnumerable<bool> data) =>
             CreateDataFrame(ToGenericRows(data), SchemaWithSingleColumn(new BooleanType()));
 
-        //TODO: Add support for System.Single (float)
-
         /// <summary>
         /// Executes a SQL query using Spark, returning the result as a DataFrame.
         /// </summary>
@@ -295,8 +285,8 @@ private StructType SchemaWithSingleColumn(DataType dataType) =>
             new StructType(new[] { new StructField("_1", dataType) });
 
         /// <summary>
-        /// Converts rows of type T to <see cref="GenericRow"/> to return a 
-        /// <see cref="IEnumerable"/> of type <see cref="GenericRow"/>.
+        /// This method is transforming each element of IEnumerable of type T input into a single 
+        /// columned GenericRow.
         /// </summary>
         /// <typeparam name="T">Datatype of values in rows</typeparam>
         /// <param name="rows">List of values of type T</param>