GitHub user brkyvz opened a pull request:

    https://github.com/apache/spark/pull/6271

    [SPARK-7745] Change asserts to requires for user input checks in Spark 
Streaming

    Assertions can be turned off. `require` throws an 
`IllegalArgumentException` which makes more sense when it's a user set variable.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/brkyvz/spark streaming-require

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/6271.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 #6271
    
----
commit afd923da42246f042f3023c6d01b7af4428a7b3f
Author: Burak Yavuz <[email protected]>
Date:   2015-05-19T21:30:15Z

    changed some assertions to require

commit 16aa7664133293ea3426a4050e200513992bc612
Author: Burak Yavuz <[email protected]>
Date:   2015-05-19T21:45:37Z

    changed more assertions to more meaningful errors

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to