attilapiros commented on a change in pull request #24205: [MINOR][DOC] Add 
missing space after comma
URL: https://github.com/apache/spark/pull/24205#discussion_r268751458
 
 

 ##########
 File path: docs/mllib-statistics.md
 ##########
 @@ -239,7 +239,7 @@ Refer to the [`Statistics` Python 
docs](api/python/pyspark.mllib.html#pyspark.ml
 ### Streaming Significance Testing
 `spark.mllib` provides online implementations of some tests to support use 
cases
 like A/B testing. These tests may be performed on a Spark Streaming
-`DStream[(Boolean,Double)]` where the first element of each tuple
+`DStream[(Boolean, Double)]` where the first element of each tuple
 
 Review comment:
   I have only found one which is not a documentation link target (where having 
no space is the correct):
   ```
   $ find . -name "*.md" -exec egrep "\[\(\w+\,\w+\)\]" \{} \;  -print
   [GraphOps.joinVertices]: 
api/scala/index.html#org.apache.spark.graphx.GraphOps@joinVertices[U](RDD[(VertexId,U)])((VertexId,VD,U)⇒VD)(ClassTag[U]):Graph[VD,ED]
   [Graph.outerJoinVertices]: 
api/scala/index.html#org.apache.spark.graphx.Graph@outerJoinVertices[U,VD2](RDD[(VertexId,U)])((VertexId,VD,Option[U])⇒VD2)(ClassTag[U],ClassTag[VD2]):Graph[VD2,ED]
   [GraphOps.collectNeighbors]: 
api/scala/index.html#org.apache.spark.graphx.GraphOps@collectNeighbors(EdgeDirection):VertexRDD[Array[(VertexId,VD)]]
   [GraphOps.pregel]: 
api/scala/index.html#org.apache.spark.graphx.GraphOps@pregel[A](A,Int,EdgeDirection)((VertexId,VD,A)⇒VD,(EdgeTriplet[VD,ED])⇒Iterator[(VertexId,A)],(A,A)⇒A)(ClassTag[A]):Graph[VD,ED]
   [Graph.apply]: 
api/scala/index.html#org.apache.spark.graphx.Graph$@apply[VD,ED](RDD[(VertexId,VD)],RDD[Edge[ED]],VD)(ClassTag[VD],ClassTag[ED]):Graph[VD,ED]
   [Graph.fromEdgeTuples]: 
api/scala/index.html#org.apache.spark.graphx.Graph$@fromEdgeTuples[VD](RDD[(VertexId,VertexId)],VD,Option[PartitionStrategy])(ClassTag[VD]):Graph[VD,Int]
         sendMsg: EdgeTriplet[VD, ED] => Iterator[(VertexId,A)],
   ./graphx-programming-guide.md
   ```
   
   So the last one: `sendMsg: EdgeTriplet[VD, ED] => Iterator[(VertexId,A)],`. 
   Which will be corrected in my next commit.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to