Skip to content

[SPARK-10973][ML][PYTHON] Fix IndexError exception on SparseVector when asked for index after the last non-zero entry#9063

Closed
zero323 wants to merge 1 commit into
apache:branch-1.4from
zero323:SPARK-10973_1.4
Closed

[SPARK-10973][ML][PYTHON] Fix IndexError exception on SparseVector when asked for index after the last non-zero entry#9063
zero323 wants to merge 1 commit into
apache:branch-1.4from
zero323:SPARK-10973_1.4

Conversation

@zero323

@zero323 zero323 commented Oct 10, 2015

Copy link
Copy Markdown
Member

See #9009 for details.

@zero323
[SPARK-10973][ML][PYTHON] Fix IndexError exception on SparseVector wh…
df2b34c 
…en asked for index after the last non-zero entry
@zero323 zero323 mentioned this pull request Oct 10, 2015
Closed
@jkbradley

jkbradley commented Oct 10, 2015

Copy link
Copy Markdown
Member

LGTM pending tests

@SparkQA

SparkQA commented Oct 10, 2015

Copy link
Copy Markdown

Test build #1873 has finished for PR 9063 at commit df2b34c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA

SparkQA commented Oct 12, 2015

Copy link
Copy Markdown

Test build #1880 has finished for PR 9063 at commit df2b34c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@jkbradley

jkbradley commented Oct 12, 2015

Copy link
Copy Markdown
Member

I tested this locally, and it worked. Merging. Thanks!

asfgit pushed a commit that referenced this pull request Oct 12, 2015
@zero323 @jkbradley
[SPARK-10973] [ML] [PYTHON] Fix IndexError exception on SparseVector …
d1cae32 
…when asked for index after the last non-zero entry

See #9009 for details.

Author: zero323 <matthew.szymkiewicz@gmail.com>

Closes #9063 from zero323/SPARK-10973_1.4.
@asfgit asfgit closed this in 615cc85 Oct 14, 2015
@zero323 zero323 deleted the SPARK-10973_1.4 branch April 6, 2017 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zero323 @jkbradley @SparkQA