Skip to content

Conversation

@jkbradley
Copy link
Member

Modified ScalaReflection.schemaFor to take primary constructor of Product when there are multiple constructors. Added test to suite which failed before but works now.

Needed for [https://github.com//pull/3637]

CC: @marmbrus

…duct when there are multiple constructors. Added test to suite which failed before but works now.
@AmplabJenkins
Copy link

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

@KirthiRaman
Copy link

unsubscribe

On Tue, Dec 9, 2014 at 1:57 PM, UCB AMPLab [email protected] wrote:

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

Reply to this email directly or view it on GitHub
#3646 (comment).

-- Kirthi Raman
http://www.linkedin.com/in/kirthiraman
(410)-302-1757 (Mobile)
(703)-444-3397 (Home)

@SparkQA
Copy link

SparkQA commented Dec 9, 2014

Test build #543 has started for PR 3646 at commit 796b2e4.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 9, 2014

Test build #543 has finished for PR 3646 at commit 796b2e4.

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

@marmbrus
Copy link
Contributor

Thanks! Merged to master.

@asfgit asfgit closed this in 2a5b5fd Dec 11, 2014
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.

5 participants