[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-07 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/18539 Merged to master --- 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

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18539 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79281/ Test PASSed. ---

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18539 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18539 **[Test build #79281 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79281/testReport)** for PR 18539 at commit

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18539 **[Test build #79281 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79281/testReport)** for PR 18539 at commit

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-06 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/18539 If you know how to display `ForeachWriter` that's passed in to `ForeachSink` nicely, let me know. `getClass.getName` didn't convince me and so I left it out. It'd be very helpful to see what

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/18539 Just happened to double check to help. Sounds we have the last one, `ForeachSink`, left. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread jaceklaskowski
Github user jaceklaskowski commented on the issue: https://github.com/apache/spark/pull/18539 I think that `ConsoleSink` was the only one with this mysterious name. We could however have another JIRA to _somehow_ unify how options are printed out for sources and sinks. I don't think

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18539 Merged build finished. Test PASSed. --- 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

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18539 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79208/ Test PASSed. ---

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18539 **[Test build #79208 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79208/testReport)** for PR 18539 at commit

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/18539 Seems OK, are there other sinks that could use a more useful toString? --- 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

[GitHub] spark issue #18539: [SPARK-21313][SS] ConsoleSink's string representation

2017-07-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18539 **[Test build #79208 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79208/testReport)** for PR 18539 at commit