Github user holdenk commented on the issue:

    https://github.com/apache/spark/pull/15496
  
    Thanks for working on this and taking the time to update to avoid the array 
copy :) There doesn't appear to be any tests for the new functionality which is 
generally a requirement and that could help us see how we would expect people 
to use this as well. Also if we make this change here we would probably want to 
something similar in `ml/linalg/` as well.



---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to