[GitHub] spark issue #20716: [SPARK-23566][Minor][Doc] Argument name mismatch fixed

2018-03-05 Thread animenon
Github user animenon commented on the issue: https://github.com/apache/spark/pull/20716 @HyukjinKwon Its minor, so may not be required. Had tagged Gator just for a check. --- - To unsubscribe, e-mail: reviews

[GitHub] spark issue #20716: [SPARK-23566][Minor][Doc] Argument name mismatch fixed

2018-03-05 Thread animenon
Github user animenon commented on the issue: https://github.com/apache/spark/pull/20716 Do check @gatorsmile --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail

[GitHub] spark pull request #20716: [SPARK-23566][Minor][Doc] Argument name mismatch ...

2018-03-03 Thread animenon
Github user animenon commented on a diff in the pull request: https://github.com/apache/spark/pull/20716#discussion_r172024390 --- Diff: python/pyspark/sql/dataframe.py --- @@ -612,9 +614,10 @@ def repartition(self, numPartitions, *cols): Returns a new :class

[GitHub] spark issue #20716: [SPARK-23566][Minor][Doc] Argument name mismatch fixed

2018-03-03 Thread animenon
Github user animenon commented on the issue: https://github.com/apache/spark/pull/20716 @HyukjinKwon Found couple more, have updated the same. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

[GitHub] spark issue #20716: [SPARK-23566][Minor][Doc] Argument name mismatch fixed

2018-03-03 Thread animenon
Github user animenon commented on the issue: https://github.com/apache/spark/pull/20716 @HyukjinKwon Sure, shall do. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail

[GitHub] spark pull request #20716: [SPARK-23566][Minor][Doc] Argument name mismatch ...

2018-03-02 Thread animenon
GitHub user animenon opened a pull request: https://github.com/apache/spark/pull/20716 [SPARK-23566][Minor][Doc] Argument name mismatch fixed Argument name mismatch fixed. ## What changes were proposed in this pull request? `col` changed to `new` in doc string

[GitHub] spark pull request #19632: Added description to python spark Pi example

2017-11-02 Thread animenon
Github user animenon commented on a diff in the pull request: https://github.com/apache/spark/pull/19632#discussion_r148628286 --- Diff: examples/src/main/python/pi.py --- @@ -27,12 +27,16 @@ if __name__ == "__main__": """

[GitHub] spark pull request #19632: Added description to python spark Pi example

2017-11-02 Thread animenon
Github user animenon commented on a diff in the pull request: https://github.com/apache/spark/pull/19632#discussion_r148620630 --- Diff: examples/src/main/python/pi.py --- @@ -27,12 +27,16 @@ if __name__ == "__main__": """

[GitHub] spark pull request #19632: Added description

2017-11-01 Thread animenon
GitHub user animenon opened a pull request: https://github.com/apache/spark/pull/19632 Added description ## What changes were proposed in this pull request? Description added to better understand example. ## How was this patch tested? Not required as only

[GitHub] spark issue #19315: [MINOR][ML]Updated english.txt word ordering

2017-09-22 Thread animenon
Github user animenon commented on the issue: https://github.com/apache/spark/pull/19315 Right, but if you want to check a few things together like "who", "who's", "why", "when", etc you will have to do many finds; but in an o

[GitHub] spark pull request #19315: Updated english.txt word ordering

2017-09-21 Thread animenon
GitHub user animenon opened a pull request: https://github.com/apache/spark/pull/19315 Updated english.txt word ordering Ordered alphabetically, for better readability. ## What changes were proposed in this pull request? Alphabetical ordering of the stop words