Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
@srowen We can check python/ml/tests.py, `VectorTests.test_serialize`
function, it contains a test for `SparseMatrix` serializing/unserializing, so
that we can confirm that this works.
---
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/14265
LGTM, merged into master. 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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14265
@holdenk or @MLnick do you happen to know if this sounds right?
---
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
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
cc @jkbradley 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
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
cc @rxin 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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14265
This isn't really my area, but I that sounds plausible
---
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
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
@srowen I guess the `_picklable_classes` list in `ml.linalg.common` is
copied from `mllib.linalg.common` so it forgot to add the `SparseMatrix` which
is added later.
---
If your project is
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
@srowen
I check the ml.python.MLSerde and it support SparseMatrix pickler and at
python side the SparseMatrix constructor also match the pickler. So I think the
`_picklable_classes`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14265
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14265
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62536/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14265
**[Test build #62536 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62536/consoleFull)**
for PR 14265 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14265
**[Test build #62536 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62536/consoleFull)**
for PR 14265 at commit
12 matches
Mail list logo
