[GitHub] spark pull request #16075: [SPARK-18516][Structured Streaming] Follow up PR ...

2016-11-29 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/16075


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #16075: [SPARK-18516][Structured Streaming] Follow up PR ...

2016-11-29 Thread tdas
Github user tdas commented on a diff in the pull request:

https://github.com/apache/spark/pull/16075#discussion_r90163483
  
--- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamingQueryListenerSuite.scala
 ---
@@ -159,28 +164,6 @@ class StreamingQueryListenerSuite extends StreamTest 
with BeforeAndAfter {
 }
   }
 
-  testQuietly("exception should be reported in QueryTerminated") {
--- End diff --

folded this test into above test.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #16075: [SPARK-18516][Structured Streaming] Follow up PR ...

2016-11-29 Thread tdas
GitHub user tdas opened a pull request:

https://github.com/apache/spark/pull/16075

[SPARK-18516][Structured Streaming] Follow up PR to add 
StreamingQuery.status to Python

## What changes were proposed in this pull request?
- Add StreamingQueryStatus.json 
- Make it not case class (to avoid unnecessarily exposing implicit object 
StreamingQueryStatus, consistent with StreamingQueryProgress)
- Add StreamingQuery.status to Python
- Fix post-termination status

## How was this patch tested?
New unit tests


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tdas/spark SPARK-18516-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/16075.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #16075


commit f823a72145d015d375eefeac47170b7906866900
Author: Tathagata Das 
Date:   2016-11-30T02:21:26Z

Added query.status to python

commit f936595019ef5fbe41df8cea5ef08d6a9c132700
Author: Tathagata Das 
Date:   2016-11-30T03:26:33Z

Fixed post termination status message




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org