[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10185 --- 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 is ena

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-10 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-163603037 Merged to master/1.6 --- 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 f

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-16011 **[Test build #2189 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2189/consoleFull)** for PR 10185 at commit [`4d82c43`](https://

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-163316582 **[Test build #2189 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2189/consoleFull)** for PR 10185 at commit [`4d82c43`](https://g

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-09 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-163315915 LGTM --- 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 a

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-08 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/10185#discussion_r47008199 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala --- @@ -892,12 +892,13 @@ object StreamingContext extends Logging {

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-08 Thread bomeng
Github user bomeng commented on a diff in the pull request: https://github.com/apache/spark/pull/10185#discussion_r46990153 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala --- @@ -892,12 +892,14 @@ object StreamingContext extends Logging {

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-162849808 **[Test build #2179 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2179/consoleFull)** for PR 10185 at commit [`21d7bcd`](https://

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-162837577 **[Test build #2179 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2179/consoleFull)** for PR 10185 at commit [`21d7bcd`](https://g

[GitHub] spark pull request: [SPARK-12136] [STREAMING] rddToFileName does n...

2015-12-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10185#discussion_r46931863 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala --- @@ -892,12 +892,14 @@ object StreamingContext extends Logging {

[GitHub] spark pull request: [SPARK-12136] [Streaming] rddToFileName does n...

2015-12-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10185#issuecomment-162749891 Can one of the admins verify this patch? --- 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 p

[GitHub] spark pull request: [Spark 12136] [Streaming] rddToFileName does n...

2015-12-07 Thread bomeng
GitHub user bomeng opened a pull request: https://github.com/apache/spark/pull/10185 [Spark 12136] [Streaming] rddToFileName does not properly handle prefix and suffix parameters The original code does not properly handle the cases where the prefix is null, but suffix is not null -