Skip to content

[SPARK-4578] fix asDict() with nested Row()#3434

Closed
davies wants to merge 1 commit intoapache:masterfrom
davies:fix_asDict
Closed

[SPARK-4578] fix asDict() with nested Row()#3434
davies wants to merge 1 commit intoapache:masterfrom
davies:fix_asDict

Conversation

@davies
Copy link
Contributor

@davies davies commented Nov 24, 2014

The Row object is created on the fly once the field is accessed, so we should access them by getattr() in asDict(0

@SparkQA
Copy link

SparkQA commented Nov 24, 2014

Test build #23798 has started for PR 3434 at commit b20f1e7.

  • This patch merges cleanly.

@JoshRosen
Copy link
Contributor

This looks good to me, pending tests. I tried it out locally in my IPython notebook and it seems to have resolved the blocker that I hit.

@SparkQA
Copy link

SparkQA commented Nov 25, 2014

Test build #23798 has finished for PR 3434 at commit b20f1e7.

  • 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/23798/
Test PASSed.

@pwendell
Copy link
Contributor

Thanks - I'm pulling this in.

@davies
Copy link
Contributor Author

davies commented Nov 25, 2014

merged

@davies davies closed this Nov 25, 2014
andrewor14 pushed a commit to andrewor14/spark that referenced this pull request Nov 25, 2014
The Row object is created on the fly once the field is accessed, so we should access them by getattr() in asDict(0

Author: Davies Liu <davies@databricks.com>

Closes apache#3434 from davies/fix_asDict and squashes the following commits:

b20f1e7 [Davies Liu] fix asDict() with nested Row()

(cherry picked from commit 050616b)
Signed-off-by: Patrick Wendell <pwendell@gmail.com>
asfgit pushed a commit that referenced this pull request Nov 25, 2014
The Row object is created on the fly once the field is accessed, so we should access them by getattr() in asDict(0

Author: Davies Liu <davies@databricks.com>

Closes #3434 from davies/fix_asDict and squashes the following commits:

b20f1e7 [Davies Liu] fix asDict() with nested Row()
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