Skip to content

Conversation

@yhuai
Copy link
Contributor

@yhuai yhuai commented Aug 1, 2014

Since we let users create Rows. It makes sense to accept mutable Maps as values of MapType columns.

JIRA: https://issues.apache.org/jira/browse/SPARK-2779

@yhuai
Copy link
Contributor Author

yhuai commented Aug 1, 2014

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 1, 2014

QA tests have started for PR 1705. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17628/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 1, 2014

QA results for PR 1705:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17628/consoleFull

@marmbrus
Copy link
Contributor

marmbrus commented Aug 1, 2014

This only failed streaming tests (cc @tdas). I'm going to merge.

@asfgit asfgit closed this in 9632719 Aug 1, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
….Map instead of scala.collection.immutable.Map

Since we let users create Rows. It makes sense to accept mutable Maps as values of MapType columns.

JIRA: https://issues.apache.org/jira/browse/SPARK-2779

Author: Yin Huai <[email protected]>

Closes apache#1705 from yhuai/SPARK-2779 and squashes the following commits:

00d72fd [Yin Huai] Use scala.collection.Map.
@yhuai yhuai deleted the SPARK-2779 branch October 6, 2014 16:26
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