Skip to content

Conversation

@mengxr
Copy link
Contributor

@mengxr mengxr commented Mar 16, 2016

What changes were proposed in this pull request?

Add row/column iterator to local matrices to simplify tasks like BlockMatrix => RowMatrix conversion. It handles dense and sparse matrices properly.

How was this patch tested?

Unit tests on sparse and dense matrix.

cc: @dbtsai

@SparkQA
Copy link

SparkQA commented Mar 16, 2016

Test build #53293 has finished for PR 11757 at commit ca5ad48.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Mar 16, 2016

Test build #53291 has finished for PR 11757 at commit 3d76781.

  • This patch passes all tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@dbtsai
Copy link
Member

dbtsai commented Mar 16, 2016

LGTM. Merged into master. Thanks.

@asfgit asfgit closed this in 85c42fd Mar 16, 2016
roygao94 pushed a commit to roygao94/spark that referenced this pull request Mar 22, 2016
## What changes were proposed in this pull request?

Add row/column iterator to local matrices to simplify tasks like BlockMatrix => RowMatrix conversion. It handles dense and sparse matrices properly.

## How was this patch tested?

Unit tests on sparse and dense matrix.

cc: dbtsai

Author: Xiangrui Meng <[email protected]>

Closes apache#11757 from mengxr/SPARK-13927.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants