GitHub user ueshin opened a pull request:

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

    [SPARK-22274][PYTHON][SQL][FOLLOWUP] Use `assertRaisesRegexp` instead of 
`assertRaisesRegex`.

    ## What changes were proposed in this pull request?
    
    This is a follow-up pr of #19872 which uses `assertRaisesRegex` but it 
doesn't exist in Python 2, so some tests fail when running tests in Python 2 
environment.
    Unfortunately, we missed it because currently Python 2 environment of the 
pr builder doesn't have proper versions of pandas or pyarrow, so the tests were 
skipped.
    
    This pr modifies to use `assertRaisesRegexp` instead of `assertRaisesRegex`.
    
    ## How was this patch tested?
    
    Tested manually in my local environment.


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

    $ git pull https://github.com/ueshin/apache-spark issues/SPARK-22274/fup1

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

    https://github.com/apache/spark/pull/20467.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 #20467
    
----
commit e9a1500be55a9b8a9affcd2513afc262cc2a666b
Author: Takuya UESHIN <ueshin@...>
Date:   2018-02-01T05:13:59Z

    Use `assertRaisesRegexp` instead of `assertRaisesRegex`.

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to