[GitHub] spark issue #21063: [SPARK-23886][Structured Streaming] Update query status ...

2018-04-23 Thread efimpoberezkin
Github user efimpoberezkin commented on the issue: https://github.com/apache/spark/pull/21063 @jose-torres Hi Jose, could you review and tell me if we need more tests or a single one will do? Also is it fine to assert status after every stream action? --- --

[GitHub] spark issue #21063: [SPARK-23886][Structured Streaming] Update query status ...

2018-04-17 Thread jose-torres
Github user jose-torres commented on the issue: https://github.com/apache/spark/pull/21063 The approach looks good to me, but we probably want to add some tests to StreamingQueryStatusAndProgressSuite. (See test("basic") in ContinuousSuite for how to set up a continuous processing mem

[GitHub] spark issue #21063: [SPARK-23886][Structured Streaming] Update query status ...

2018-04-17 Thread efimpoberezkin
Github user efimpoberezkin commented on the issue: https://github.com/apache/spark/pull/21063 Hi @jose-torres, I made some changes to this PR according to your comment, could you review it please? --- - To unsubscri