Skip to content

[SPARK-12058][Hotfix]Disable KinesisStreamTests#10047

Closed
zsxwing wants to merge 2 commits into
apache:masterfrom
zsxwing:disable-python-kinesis-test
Closed

[SPARK-12058][Hotfix]Disable KinesisStreamTests#10047
zsxwing wants to merge 2 commits into
apache:masterfrom
zsxwing:disable-python-kinesis-test

Conversation

@zsxwing

@zsxwing zsxwing commented Nov 30, 2015

Copy link
Copy Markdown
Member

KinesisStreamTests in test.py is broken because of #9403. See https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46896/testReport/(root)/KinesisStreamTests/test_kinesis_stream/

Because Streaming Python didn’t work when merging #9403, the PR build didn’t report the Python test failure actually.

This PR just disabled the test to unblock #10039

@zsxwing

zsxwing commented Nov 30, 2015

Copy link
Copy Markdown
Member Author

cc @mengxr @brkyvz @tdas

@mengxr

mengxr commented Nov 30, 2015

Copy link
Copy Markdown
Contributor

@zsxwing Could you create a JIRA so we remember to add the test back?

@zsxwing

zsxwing commented Nov 30, 2015

Copy link
Copy Markdown
Member Author

@zsxwing Could you create a JIRA so we remember to add the test back?

sure. https://issues.apache.org/jira/browse/SPARK-12058

@SparkQA

SparkQA commented Nov 30, 2015

Copy link
Copy Markdown

Test build #46903 has finished for PR 10047 at commit 765b584.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class ChainedIterator extends UnsafeSorterIterator\n * class UnsafeCartesianRDD(left : RDD[UnsafeRow], right : RDD[UnsafeRow], numFieldsOfRight: Int)\n

Comment thread python/pyspark/streaming/tests.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mention SPARK-12058 in the message.

@mengxr

mengxr commented Nov 30, 2015

Copy link
Copy Markdown
Contributor

Could you add [SPAKR-12058] in the PR title?

@zsxwing zsxwing changed the title [Hotfix]Disable KinesisStreamTests [SPAKR-12058][Hotfix]Disable KinesisStreamTests Nov 30, 2015
@zsxwing

zsxwing commented Nov 30, 2015

Copy link
Copy Markdown
Member Author

Updated

@SparkQA

SparkQA commented Nov 30, 2015

Copy link
Copy Markdown

Test build #46909 has finished for PR 10047 at commit c514866.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class ChainedIterator extends UnsafeSorterIterator\n * class UnsafeCartesianRDD(left : RDD[UnsafeRow], right : RDD[UnsafeRow], numFieldsOfRight: Int)\n

@zsxwing zsxwing changed the title [SPAKR-12058][Hotfix]Disable KinesisStreamTests [SPARK-12058][Hotfix]Disable KinesisStreamTests Dec 1, 2015
@zsxwing

zsxwing commented Dec 1, 2015

Copy link
Copy Markdown
Member Author

Merging to master to unblock #10039

@asfgit asfgit closed this in edb26e7 Dec 1, 2015
@zsxwing zsxwing deleted the disable-python-kinesis-test branch December 1, 2015 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants