Github user jkbradley commented on the issue:

    https://github.com/apache/spark/pull/20088
  
    This is hopefully a rare case of an error in assuming an ordering of rows; 
in general, we shouldn't (and don't, AFAIK) assume an order for rows in 
saving/loading models.
    
    LGTM, but before this is merged, can you please remove the ML and CORE tags 
from the PR title @WeichenXu123 ?


---

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

Reply via email to