Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/17296
@MLnick , I found an existing `MLTestingUtils.checkCopy` that is used to
check the copied model uids match and can easily be extended to include the
check needed here. I went through and added these checks to any ML suite that
wasn't already using, but that led to another issue that I felt should be
covered in a separate PR at #17326. Can you take a look at that first and
merge if ok, then I'll update this push the regression tests for this? Thanks!
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
