addressed comments v0.1

brkyvz · brkyvz · commit 4fe693b6d1bd · 2015-05-01T21:54:33.000-07:00
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala b/sql/core/src/main/scala/org/apache/spark/sql/DataFrameStatFunctions.scala
@@ -28,7 +28,7 @@ import org.apache.spark.sql.execution.stat._
 final class DataFrameStatFunctions private[sql](df: DataFrame) {
 
   /**
-   * Calculate the correlation of two columns of a DataFrame. Currently only supports the Pearson
+   * Calculates the correlation of two columns of a DataFrame. Currently only supports the Pearson
    * Correlation Coefficient. For Spearman Correlation, consider using RDD methods found in 
    * MLlib's Statistics.
    *
@@ -43,7 +43,7 @@ final class DataFrameStatFunctions private[sql](df: DataFrame) {
   }
 
   /**
-   * Java Friendly implementation to calculate the Pearson correlation coefficient of two columns.
+   * Calculates the Pearson Correlation Coefficient of two columns of a DataFrame.
    *
    * @param col1 the name of the column
    * @param col2 the name of the column to calculate the correlation against

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ import org.apache.spark.sql.execution.stat._`
`28`	`28`	`final class DataFrameStatFunctions private[sql](df: DataFrame) {`
`29`	`29`
`30`	`30`	`/**`
`31`		`- * Calculate the correlation of two columns of a DataFrame. Currently only supports the Pearson`
	`31`	`+ * Calculates the correlation of two columns of a DataFrame. Currently only supports the Pearson`
`32`	`32`	`* Correlation Coefficient. For Spearman Correlation, consider using RDD methods found in`
`33`	`33`	`* MLlib's Statistics.`
`34`	`34`	`*`
`@@ -43,7 +43,7 @@ final class DataFrameStatFunctions private[sql](df: DataFrame) {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`/**`
`46`		`- * Java Friendly implementation to calculate the Pearson correlation coefficient of two columns.`
	`46`	`+ * Calculates the Pearson Correlation Coefficient of two columns of a DataFrame.`
`47`	`47`	`*`
`48`	`48`	`* @param col1 the name of the column`
`49`	`49`	`* @param col2 the name of the column to calculate the correlation against`