-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-8564][Streaming]Add the Python API for Kinesis #6955
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
python/run-tests
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to move this logic into pyspark/streaming/test.py's main method instead? I'm in the middle of a Python test runner refactoring and it would greatly simplify things if the test suites themselves enforced that their prerequisites were installed rather than pushing that responsibility into the test triggering script.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw your refactoring work. I will update it once your PR is merged.
|
Test build #35554 has finished for PR 6955 at commit
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version is never used in the error message. This fails with:
TypeError: not all arguments converted during string formatting
|
Test build #38223 has finished for PR 6955 at commit
|
|
Test build #38230 has finished for PR 6955 at commit
|
|
Test build #38236 has finished for PR 6955 at commit
|
|
Test build #38312 has finished for PR 6955 at commit
|
|
Test build #38341 has finished for PR 6955 at commit
|
|
Test build #38353 has finished for PR 6955 at commit
|
|
Test build #38416 has finished for PR 6955 at commit
|
|
Test build #38426 has finished for PR 6955 at commit
|
|
retest this please |
|
Test build #100 has finished for PR 6955 at commit
|
|
Test build #38433 has finished for PR 6955 at commit
|
|
Test build #38449 timed out for PR 6955 at commit |
|
retest this please |
|
Test build #104 has finished for PR 6955 at commit
|
|
@zsxwing could you update this PR by merging with the master. And you probably need to add the environment variable thing to the new module. |
…e folder to streaming_kinesis_asl module
|
Test build #39028 has finished for PR 6955 at commit
|
|
Jenkins, test this please. |
|
Test build #39032 has finished for PR 6955 at commit
|
|
Test build #39038 has finished for PR 6955 at commit
|
|
Test build #1240 has finished for PR 6955 at commit
|
|
LGTM. I will merge this when this build passes. |
|
Test build #39127 timed out for PR 6955 at commit |
|
retest this please |
|
Test build #173 has finished for PR 6955 at commit
|
|
Test build #39149 has finished for PR 6955 at commit
|
|
retest this please |
|
Test build #174 has started for PR 6955 at commit |
|
Test build #39175 has finished for PR 6955 at commit
|
|
retest this please |
|
Test build #1255 has finished for PR 6955 at commit
|
|
retest this please |
|
Test build #176 has finished for PR 6955 at commit
|
|
Test build #39201 has finished for PR 6955 at commit
|
|
Test build #177 has finished for PR 6955 at commit
|
|
Test build #39205 has finished for PR 6955 at commit
|
|
Yeah, it does not. Other tests passed, so I am merging this to master. Thanks! |
This PR adds the Python API for Kinesis, including a Python example and a simple unit test.