Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/11030#discussion_r51634167
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/StreamTest.scala ---
@@ -179,16 +206,38 @@ trait StreamTest extends QueryTest with Timeouts {
|${if (streamDeathCause != null)
stackTraceToString(streamDeathCause) else ""}
|
|== Sink ==
- |$sink
+ |${sink.toLongString}
|
|== Plan ==
|${if (currentStream != null) currentStream.lastExecution else ""}
- """
+ """.stripMargin
+
+ def assert(condition: => Boolean, message: String): Unit = {
--- End diff --
These are a bunch of changes to make sure that any kind of `assert` and
`eventually` that is needed within `testStream` is wrapped in a try catch so
that we can catch them and enrich the message with the `testState`.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
