Skip to content

Conversation

@gzm0
Copy link
Contributor

@gzm0 gzm0 commented Feb 6, 2015

No description provided.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@gzm0
Copy link
Contributor Author

gzm0 commented Feb 6, 2015

Review by @marmbrus

@marmbrus
Copy link
Contributor

marmbrus commented Feb 6, 2015

ok to test

@SparkQA
Copy link

SparkQA commented Feb 6, 2015

Test build #26923 has started for PR 4431 at commit c5da21e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 6, 2015

Test build #26923 has finished for PR 4431 at commit c5da21e.

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

@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/26923/
Test PASSed.

@marmbrus
Copy link
Contributor

marmbrus commented Feb 6, 2015

Thanks! Merged to master and 1.3

asfgit pushed a commit that referenced this pull request Feb 6, 2015
Author: Tobias Schlatter <tobias@meisch.ch>

Closes #4431 from gzm0/sync-scala-refl and squashes the following commits:

c5da21e [Tobias Schlatter] [SPARK-5640] Synchronize ScalaReflection where necessary

(cherry picked from commit 500dc2b)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 500dc2b Feb 6, 2015
@gzm0 gzm0 deleted the sync-scala-refl branch February 12, 2015 06:19
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