You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by tdas <gi...@git.apache.org> on 2018/02/21 11:13:35 UTC
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
GitHub user tdas opened a pull request:
https://github.com/apache/spark/pull/20650
[SPARK-23408][SS] Synchronize successive AddData actions in Streaming*JoinSuite
## What changes were proposed in this pull request?
The stream-stream join tests add data to multiple sources and expect it all to show up in the next batch. But there's a race condition; the new batch might trigger when only one of the AddData actions has been reached.
Prior attempt to solve this issue by @jose-torres in #20646 attempted to simultaneously synchronize on all memory sources together when consecutive AddData was found in the actions. However, this carries the risk of deadlock as well as unintended modification of stress tests (see the above PR for a detailed explanation). Instead, this PR attempts the following.
- A new action called `StreamProgressBlockedActions` that allows multiple actions to be executed while the streaming query is blocked from making progress. This allows data to be added to multiple sources that are made visible simultaneously in the next batch.
- An alias of `StreamProgressBlockedActions` called `MultiAddData` is explicitly used in the `Streaming*JoinSuites` to add data to two memory sources simultaneously.
## How was this patch tested?
Modified test cases in `Streaming*JoinSuites` where there are consecutive `AddData` actions.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tdas/spark SPARK-23408
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/20650.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 #20650
----
commit b4c3c55db394178f083d3eeaf537e407c026f0cd
Author: Tathagata Das <ta...@...>
Date: 2018-02-21T10:48:15Z
Fixed bug
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4104 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4104/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by jose-torres <gi...@git.apache.org>.
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20650
LGTM
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/986/
Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20650
All the failures above can be attributed to other flakiness unrelated to the flakiness this PR trying to address.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87585/
Test FAILed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4111 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4111/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20650#discussion_r169605952
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamTest.scala ---
@@ -425,243 +444,248 @@ trait StreamTest extends QueryTest with SharedSQLContext with TimeLimits with Be
}
}
- var manualClockExpectedTime = -1L
- val defaultCheckpointLocation =
- Utils.createTempDir(namePrefix = "streaming.metadata").getCanonicalPath
- try {
- startedTest.foreach { action =>
- logInfo(s"Processing test stream action: $action")
- action match {
- case StartStream(trigger, triggerClock, additionalConfs, checkpointLocation) =>
- verify(currentStream == null, "stream already running")
- verify(triggerClock.isInstanceOf[SystemClock]
- || triggerClock.isInstanceOf[StreamManualClock],
- "Use either SystemClock or StreamManualClock to start the stream")
- if (triggerClock.isInstanceOf[StreamManualClock]) {
- manualClockExpectedTime = triggerClock.asInstanceOf[StreamManualClock].getTimeMillis()
+ def performAction(action: StreamAction): Unit = {
--- End diff --
TODO: Add docs.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20650#discussion_r169604484
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala ---
@@ -543,6 +543,15 @@ abstract class StreamExecution(
Option(name).map(_ + "<br/>").getOrElse("") +
s"id = $id<br/>runId = $runId<br/>batch = $batchDescription"
}
+
+ private[sql] def withProgressLocked(f: => Unit): Unit = {
--- End diff --
TODO: Add docs.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4119 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4119/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4116 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4116/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4107 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4107/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4113 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4113/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4108 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4108/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4106 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4106/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4105 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4105/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/994/
Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4111 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4111/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87596/
Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4110 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4110/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4110 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4110/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4114 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4114/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20650#discussion_r169605919
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamTest.scala ---
@@ -217,6 +225,14 @@ trait StreamTest extends QueryTest with SharedSQLContext with TimeLimits with Be
s"ExpectFailure[${causeClass.getName}, isFatalError: $isFatalError]"
}
+ case class StreamProgressLockedActions(actions: Seq[StreamAction], desc: String = null)
--- End diff --
TODO: add docs.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4120 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4120/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4106 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4106/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87597/
Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4125 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4125/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by jose-torres <gi...@git.apache.org>.
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20650
I'm not sure I agree with all the comments on the previous PR, but I agree that this also works.
As discussed, the downside to this approach is that people in the future can continue to write the same kind of flaky tests this PR fixes. Ideally I'd like to see some kind of story for how people will know they must use MultiAddData.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4107 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4107/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4120 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4120/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4124 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4124/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4105 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4105/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4123 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4123/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/20650#discussion_r169605887
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamTest.scala ---
@@ -102,6 +102,14 @@ trait StreamTest extends QueryTest with SharedSQLContext with TimeLimits with Be
AddDataMemory(source, data)
}
+ object MultiAddData {
--- End diff --
TODO: add docs.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4108 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4108/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87597 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87597/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4116 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4116/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87596 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87596/testReport)** for PR 20650 at commit [`7b78fa1`](https://github.com/apache/spark/commit/7b78fa12dc0203ecdcf85a6c88effae37008bf28).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4119 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4119/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark pull request #20650: [SPARK-23408][SS] Synchronize successive AddData ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20650
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4123 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4123/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails PySpark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87585 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87585/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by zsxwing <gi...@git.apache.org>.
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20650
lgtm
On Thu, Feb 22, 2018 at 3:38 PM Jose Torres <no...@github.com>
wrote:
> LGTM
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/spark/pull/20650#issuecomment-367860182>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/AA9FSiW0uIGqfWt3RjsyoY2p7x_B-ClHks5tXfptgaJpZM4SNdOS>
> .
>
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4122 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4122/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4118 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4118/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87596 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87596/testReport)** for PR 20650 at commit [`7b78fa1`](https://github.com/apache/spark/commit/7b78fa12dc0203ecdcf85a6c88effae37008bf28).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/995/
Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87597 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87597/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20650
@zsxwing can you also take a look?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4114 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4114/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4113 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4113/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4117 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4117/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4117 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4117/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4121 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4121/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4109 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4109/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4124 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4124/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4112 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4112/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #87585 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87585/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4122 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4122/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4109 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4109/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4125 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4125/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
* This patch **fails Spark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by tdas <gi...@git.apache.org>.
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/20650
Yes, this is indeed a slight downside. The only time people should choose to use it if they want to add data in multiple sources that are to be visible in the batch. In our case, we need to add data in multiple sources in the same batch because we want to verify the number of state rows changed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4104 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4104/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test PASSed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20650
Merged build finished. Test FAILed.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4112 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4112/testReport)** for PR 20650 at commit [`b4c3c55`](https://github.com/apache/spark/commit/b4c3c55db394178f083d3eeaf537e407c026f0cd).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4121 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4121/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] spark issue #20650: [SPARK-23408][SS] Synchronize successive AddData actions...
Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20650
**[Test build #4118 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4118/testReport)** for PR 20650 at commit [`fdcf716`](https://github.com/apache/spark/commit/fdcf716b59a6c6825a01309973f7f053374cf620).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org