[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-05-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12521 --- 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

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-05-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-216397719 Thanks, merging to master and 2.0 --- 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

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214678357 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214678356 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

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214678058 **[Test build #56985 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56985/consoleFull)** for PR 12521 at commit

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214657972 **[Test build #56985 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56985/consoleFull)** for PR 12521 at commit

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-26 Thread lw-lin
Github user lw-lin commented on a diff in the pull request: https://github.com/apache/spark/pull/12521#discussion_r61041131 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/streaming/DataFrameReaderWriterSuite.scala --- @@ -368,4 +368,79 @@ class DataFrameReaderWriterSuite

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-25 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214585357 Updates: thanks to [[SPARK-14473][SQL] Define analysis rules to catch operations not supported in

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-25 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-214482001 /cc @tdas --- 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 pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/12521#discussion_r60969585 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/streaming/DataFrameReaderWriterSuite.scala --- @@ -368,4 +368,79 @@ class

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-212336163 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-212336157 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

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-212335403 **[Test build #56336 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56336/consoleFull)** for PR 12521 at commit

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-212303861 @rxin @marmbrus would you mind taking a look when you have time? Thanks! :-) And I'm not sure we should disallow calling methods like `parquet()`, `text()` on

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12521#issuecomment-212299036 **[Test build #56336 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56336/consoleFull)** for PR 12521 at commit

[GitHub] spark pull request: [SPARK-14747][SQL] Add assertStreaming/assertN...

2016-04-20 Thread lw-lin
GitHub user lw-lin opened a pull request: https://github.com/apache/spark/pull/12521 [SPARK-14747][SQL] Add assertStreaming/assertNoneStreaming checks in DataFrameWriter ## Problem If an end user happens to write code mixed with continuous-query-oriented methods and