huaxingao opened a new pull request #27204: [MINOR][ML] Change 
DecisionTreeClassifier to FMClassifier in OneVsRest setWeightCol test
URL: https://github.com/apache/spark/pull/27204
 
 
   
   ### What changes were proposed in this pull request?
   Change ```DecisionTreeClassifier``` to ```FMClassifier``` in ```OneVsRest``` 
setWeightCol test 
   
   ### Why are the changes needed?
   In ```OneVsRest```, if the classifier doesn't support instance weight, 
```OneVsRest``` weightCol will be ignored, so unit test has tested one 
classifier(```LogisticRegression```) that support instance weight, and one 
classifier (```DecisionTreeClassifier```) that doesn't support instance weight. 
Since ```DecisionTreeClassifier``` now supports instance weight, we need to 
change it to the classifier that doesn't have weight support. 
   
   
   ### Does this PR introduce any user-facing change?
   No
   
   
   ### How was this patch tested?
   Existing test
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to