[GitHub] spark pull request #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-15 Thread linbojin
Github user linbojin closed the pull request at: https://github.com/apache/spark/pull/16276 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] spark issue #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-15 Thread linbojin
Github user linbojin commented on the issue: https://github.com/apache/spark/pull/16276 @rxin Thx, I just deep dive into spark, I hope i can try to contribute more impactful things later 😄 . I will close this one. @srowen I tried out your idea that i created

[GitHub] spark issue #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-15 Thread linbojin
Github user linbojin commented on the issue: https://github.com/apache/spark/pull/16276 @srowen So any problems with my current implementation, i mean use `implicit asTraversable: T => TraversableOnce[U]`. Because I refer to `flatten` implementation in scala source code: ht

[GitHub] spark pull request #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-14 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/16276#discussion_r92550699 --- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala --- @@ -381,6 +381,14 @@ abstract class RDD[T: ClassTag

[GitHub] spark pull request #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-14 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/16276#discussion_r92546374 --- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala --- @@ -381,6 +381,14 @@ abstract class RDD[T: ClassTag

[GitHub] spark pull request #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-14 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/16276#discussion_r92531327 --- Diff: core/src/test/scala/org/apache/spark/rdd/RDDSuite.scala --- @@ -88,6 +88,13 @@ class RDDSuite extends SparkFunSuite with SharedSparkContext

[GitHub] spark issue #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-14 Thread linbojin
Github user linbojin commented on the issue: https://github.com/apache/spark/pull/16276 cc @rxin --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] spark pull request #16276: [SPARK-18855][CORE] Add RDD flatten function

2016-12-13 Thread linbojin
GitHub user linbojin opened a pull request: https://github.com/apache/spark/pull/16276 [SPARK-18855][CORE] Add RDD flatten function ## What changes were proposed in this pull request? Added a new flatten function for RDD. ## How was this patch tested? Unit

[GitHub] spark pull request #16132: [MINOR] [README] Correct Markdown link inside rea...

2016-12-03 Thread linbojin
GitHub user linbojin opened a pull request: https://github.com/apache/spark/pull/16132 [MINOR] [README] Correct Markdown link inside readme ## What changes were proposed in this pull request? "Useful Developer Tools" link inside [README.md](https://github.com/ap

[GitHub] spark pull request #14645: [MINOR] [DOC] Correct code snippet results in qui...

2016-08-15 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/14645#discussion_r74737989 --- Diff: docs/quick-start.md --- @@ -40,7 +40,7 @@ RDDs have _[actions](programming-guide.html#actions)_, which return values, and {% highlight

[GitHub] spark pull request #14645: [MINOR] [DOC] Correct code snippet results in qui...

2016-08-15 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/14645#discussion_r74737117 --- Diff: docs/quick-start.md --- @@ -40,7 +40,7 @@ RDDs have _[actions](programming-guide.html#actions)_, which return values, and {% highlight

[GitHub] spark pull request #14645: [MINOR] [DOC] Correct code snippet results in qui...

2016-08-15 Thread linbojin
Github user linbojin commented on a diff in the pull request: https://github.com/apache/spark/pull/14645#discussion_r74733368 --- Diff: docs/quick-start.md --- @@ -40,7 +40,7 @@ RDDs have _[actions](programming-guide.html#actions)_, which return values, and {% highlight

[GitHub] spark pull request #14645: [MINOR] [DOC] Correct code snippet results in qui...

2016-08-15 Thread linbojin
GitHub user linbojin opened a pull request: https://github.com/apache/spark/pull/14645 [MINOR] [DOC] Correct code snippet results in quick start documentation ## What changes were proposed in this pull request? As README.md file is updated over time. Some code snippet

[GitHub] spark pull request: [MINOR][SQL][DOCS] Add notes of the determinis...

2016-05-23 Thread linbojin
Github user linbojin commented on the pull request: https://github.com/apache/spark/pull/13087#issuecomment-221148743 @marmbrus @dongjoon-hyun I will add the detail description to the old SPARK-15282 JIRA issue. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-15282][SQL] PushDownPredicate should no...

2016-05-22 Thread linbojin
Github user linbojin commented on the pull request: https://github.com/apache/spark/pull/13087#issuecomment-220837854 @dongjoon-hyun @cloud-fan @marmbrus Thanks for your discussions about my reported issue: [SPARK-15282](https://issues.apache.org/jira/browse/SPARK-15282

[GitHub] spark pull request: [SPARK-15282][SQL] PushDownPredicate should no...

2016-05-19 Thread linbojin
Github user linbojin commented on the pull request: https://github.com/apache/spark/pull/13087#issuecomment-220502400 Hi, @marmbrus Could you review this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your