[GitHub] spark pull request: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8257 --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132408447 **[Test build #1654 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1654/console)** for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206) after a configured wait of `175m`. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132406545 [Test build #1662 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1662/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class FilterNode(condition: Expression, child: LocalNode) extends UnaryLocalNode ` * `abstract class LocalNode extends TreeNode[LocalNode] ` * `abstract class LeafLocalNode extends LocalNode ` * `abstract class UnaryLocalNode extends LocalNode ` * `case class ProjectNode(projectList: Seq[NamedExpression], child: LocalNode) extends UnaryLocalNode ` * `case class SeqScanNode(output: Seq[Attribute], data: Seq[InternalRow]) extends LeafLocalNode ` * `public final class UTF8String implements Comparable, Externalizable ` --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132403153 Merging this to master and 1.5. Thanks @zsxwing and @harishreedharan for reviewing this. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132402007 [Test build #1658 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1658/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132395694 [Test build #1662 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1662/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132390907 [Test build #1658 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1658/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132378041 [Test build #1653 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1653/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132372446 [Test build #1654 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1654/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132369841 [Test build #1652 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1652/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132368265 [Test build #1653 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1653/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132356303 [Test build #1652 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1652/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132355177 [Test build #1651 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1651/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132342208 [Test build #1651 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1651/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132192872 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41119/ Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132192869 Merged build finished. Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132192795 **[Test build #41119 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41119/console)** for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206) after a configured wait of `175m`. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132185583 [Test build #41122 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41122/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132185923 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41122/ 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 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132185922 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 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132170849 [Test build #41122 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41122/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132169325 Merged build started. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132169275 Merged build triggered. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132167980 retest this please --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132160360 **[Test build #1647 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1647/console)** for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b) after a configured wait of `175m`. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132142836 [Test build #41119 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41119/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132140657 Merged build started. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132140589 Merged build triggered. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132137963 retest this please --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132118501 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41103/ Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132118499 Merged build finished. Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132118456 [Test build #41103 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41103/console) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132108499 [Test build #41103 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41103/consoleFull) for PR 8257 at commit [`cb7bba2`](https://github.com/apache/spark/commit/cb7bba2f3ba1f3af87a55f2fc4f38da142099206). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132105673 Merged build triggered. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132105745 Merged build started. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132105370 [Test build #1647 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1647/consoleFull) for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132105272 LGTM --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8257#discussion_r37271032 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala --- @@ -227,16 +227,21 @@ private[streaming] class BlockGenerator( def isStopped(): Boolean = state == StoppedAll /** Change the buffer to which single records are added to. */ - private def updateCurrentBuffer(time: Long): Unit = synchronized { + private def updateCurrentBuffer(time: Long): Unit = { try { - val newBlockBuffer = currentBuffer - currentBuffer = new ArrayBuffer[Any] - if (newBlockBuffer.size > 0) { -val blockId = StreamBlockId(receiverId, time - blockIntervalMs) -val newBlock = new Block(blockId, newBlockBuffer) -listener.onGenerateBlock(blockId) + var newBlock: Block = null + synchronized { +if (currentBuffer.nonEmpty) { + val newBlockBuffer = currentBuffer + currentBuffer = new ArrayBuffer[Any] + val blockId = StreamBlockId(receiverId, time - blockIntervalMs) + listener.onGenerateBlock(blockId) + newBlock = new Block(blockId, newBlockBuffer) +} + } + + if (newBlock != null) { --- End diff -- Sorry that I misunderstood the meaning of `state`. This code is right. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132102583 LGTM --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/8257#discussion_r37267126 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala --- @@ -227,16 +227,21 @@ private[streaming] class BlockGenerator( def isStopped(): Boolean = state == StoppedAll /** Change the buffer to which single records are added to. */ - private def updateCurrentBuffer(time: Long): Unit = synchronized { + private def updateCurrentBuffer(time: Long): Unit = { try { - val newBlockBuffer = currentBuffer - currentBuffer = new ArrayBuffer[Any] - if (newBlockBuffer.size > 0) { -val blockId = StreamBlockId(receiverId, time - blockIntervalMs) -val newBlock = new Block(blockId, newBlockBuffer) -listener.onGenerateBlock(blockId) + var newBlock: Block = null + synchronized { +if (currentBuffer.nonEmpty) { + val newBlockBuffer = currentBuffer + currentBuffer = new ArrayBuffer[Any] + val blockId = StreamBlockId(receiverId, time - blockIntervalMs) + listener.onGenerateBlock(blockId) + newBlock = new Block(blockId, newBlockBuffer) +} + } + + if (newBlock != null) { --- End diff -- The condition regarding state change can be done. But did you suggest that to solve the race condition? Because I dont think it solves it. I dont think that the timer thread that calls `updateCurrentBuffer` will be interrupted as the timer is stopped with [interruptTimer = false](https://github.com/apache/spark/blob/master/streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala#L145). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8257#discussion_r37260735 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala --- @@ -227,16 +227,21 @@ private[streaming] class BlockGenerator( def isStopped(): Boolean = state == StoppedAll /** Change the buffer to which single records are added to. */ - private def updateCurrentBuffer(time: Long): Unit = synchronized { + private def updateCurrentBuffer(time: Long): Unit = { try { - val newBlockBuffer = currentBuffer - currentBuffer = new ArrayBuffer[Any] - if (newBlockBuffer.size > 0) { -val blockId = StreamBlockId(receiverId, time - blockIntervalMs) -val newBlock = new Block(blockId, newBlockBuffer) -listener.onGenerateBlock(blockId) + var newBlock: Block = null + synchronized { +if (currentBuffer.nonEmpty) { + val newBlockBuffer = currentBuffer + currentBuffer = new ArrayBuffer[Any] + val blockId = StreamBlockId(receiverId, time - blockIntervalMs) + listener.onGenerateBlock(blockId) + newBlock = new Block(blockId, newBlockBuffer) +} + } + + if (newBlock != null) { --- End diff -- How about changing ``` def isGeneratingBlocks = synchronized { state == Active || state == StoppedAddingData } ``` to ``` def isGeneratingBlocks = synchronized { state == Active || state == StoppedGeneratingBlocks } ``` ? --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/8257#discussion_r37260487 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/receiver/BlockGenerator.scala --- @@ -227,16 +227,21 @@ private[streaming] class BlockGenerator( def isStopped(): Boolean = state == StoppedAll /** Change the buffer to which single records are added to. */ - private def updateCurrentBuffer(time: Long): Unit = synchronized { + private def updateCurrentBuffer(time: Long): Unit = { try { - val newBlockBuffer = currentBuffer - currentBuffer = new ArrayBuffer[Any] - if (newBlockBuffer.size > 0) { -val blockId = StreamBlockId(receiverId, time - blockIntervalMs) -val newBlock = new Block(blockId, newBlockBuffer) -listener.onGenerateBlock(blockId) + var newBlock: Block = null + synchronized { +if (currentBuffer.nonEmpty) { + val newBlockBuffer = currentBuffer + currentBuffer = new ArrayBuffer[Any] + val blockId = StreamBlockId(receiverId, time - blockIntervalMs) + listener.onGenerateBlock(blockId) + newBlock = new Block(blockId, newBlockBuffer) +} + } + + if (newBlock != null) { --- End diff -- Assume the thread of `updateCurrentBuffer` is preempted here (we can add a Thread.sleep(1) to emulate it) while BlockGenerator is stopping, looks it's possible that `newBlock` won't be polled. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132035689 LGTM2 --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132030842 [Test build #1646 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1646/console) for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132029262 [Test build #41081 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41081/console) for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b). * This patch **fails PySpark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132029418 Merged build finished. Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132029424 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41081/ Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132014215 [Test build #41081 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41081/consoleFull) for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132013835 Merged build triggered. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132013906 Merged build started. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132013151 [Test build #1646 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1646/consoleFull) for PR 8257 at commit [`fcb44ce`](https://github.com/apache/spark/commit/fcb44ce68a384b50b2be827f7bb607b7c02c282b). --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-132012109 Jenkins, test this please. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user harishreedharan commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-131998349 +1. LGTM --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-131995394 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41069/ Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-131995391 Merged build finished. Test FAILed. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-131993710 Merged build triggered. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8257#issuecomment-131993727 Merged build started. --- 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: [SPARK-10072][STREAMING] BlockGenerator can de...
GitHub user tdas opened a pull request: https://github.com/apache/spark/pull/8257 [SPARK-10072][STREAMING] BlockGenerator can deadlock when the queue block queue of generate blocks fills up to capacity Moved blocking call to ArrayBlockingQueue outside the synchronized to prevent deadlock. You can merge this pull request into a Git repository by running: $ git pull https://github.com/tdas/spark SPARK-10072 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/8257.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 #8257 commit fcb44ce68a384b50b2be827f7bb607b7c02c282b Author: Tathagata Das Date: 2015-08-17T23:26:41Z Moved blocking call to ArrayBlockingQueue outside the synchronized to prevent deadlock. --- 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