Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Feb 9, 2015

Fix ARPACK error code mapping, at least. It's not yet clear whether the error is what we expect from ARPACK. If it isn't, not sure if that's to be treated as an MLlib or Breeze issue.

@SparkQA
Copy link

SparkQA commented Feb 9, 2015

Test build #27136 has started for PR 4485 at commit 7355aa1.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27136 has finished for PR 4485 at commit 7355aa1.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class MasterStateResponse(
    • class LocalSparkCluster(
    • * (4) the main class for the child
    • case class BoundPortsResponse(actorPort: Int, webUIPort: Int, restPort: Option[Int])
    • throw new SubmitRestMissingFieldException("Main class is missing.")
    • public class JavaLDAExample

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27136/
Test PASSed.

asfgit pushed a commit that referenced this pull request Feb 10, 2015
…teException

Fix ARPACK error code mapping, at least. It's not yet clear whether the error is what we expect from ARPACK. If it isn't, not sure if that's to be treated as an MLlib or Breeze issue.

Author: Sean Owen <sowen@cloudera.com>

Closes #4485 from srowen/SPARK-4900 and squashes the following commits:

7355aa1 [Sean Owen] Fix ARPACK error code mapping

(cherry picked from commit 36c4e1d)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
@asfgit asfgit closed this in 36c4e1d Feb 10, 2015
@mengxr
Copy link
Contributor

mengxr commented Feb 10, 2015

LGTM. Merged into master and branch-1.3. Thanks! (Please re-open the JIRA if the issue is not fully resolved.)

@srowen srowen deleted the SPARK-4900 branch February 10, 2015 09:12
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.

4 participants