GitHub user shahidki31 opened a pull request:

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

    [Minor][ML]Added UT for checking maximum number of features for 
GeneralizedLinearRegression and WeightedLeastSquares

    Currently in the GeneralizedLinearRegression and WeightedLeastSquare 
doesn't support  more than 4096 features. But there is no UT added to check 
this. In this PR, I have added UT for checking the behavior of both the 
algorithms, if the features are more than 4096.
    
    ## How was this patch tested?
    Added UT
    


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

    $ git pull https://github.com/shahidki31/spark maxFeatureCheckUT

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

    https://github.com/apache/spark/pull/21842.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 #21842
    
----
commit 42c61eaeecff0c37fee1be35ba48494b70639b70
Author: Shahid <shahidki31@...>
Date:   2018-07-22T19:28:08Z

    [Minor][ML]Added UT for checking maximum number of features for 
GeneralizedLinearRegression and WeightedLeastSquares
    
    Currently GeneralizedLinearRegression and WeightedLeastSquare doesn't 
support features more than 4096. But there is no UT added to check this. In 
this PR, I have added UT for checking the behaviour of both the algorithms, if 
the features are more than 4096.

----


---

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

Reply via email to