New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-23588][SQL][FOLLOW-UP] Resolve a map builder method per execution in CatalystToExternalMap #21112

Closed

maropu wants to merge 1 commit into apache:master from maropu:SPARK-23588-FOLLOWUP

Member

maropu commented Apr 20, 2018

What changes were proposed in this pull request?

This pr is a follow-up pr of #20979 and fixes code to resolve a map builder method per execution instead of per row in CatalystToExternalMap.

How was this patch tested?

Existing tests.

Fix

1474cd9

Member Author

maropu commented Apr 20, 2018

SparkQA commented Apr 20, 2018

Test build #89609 has finished for PR 21112 at commit 1474cd9.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Member Author

maropu commented Apr 20, 2018

retest this please

Contributor

hvanhovell commented Apr 20, 2018

LGTM

SparkQA commented Apr 20, 2018

Test build #89618 has finished for PR 21112 at commit 1474cd9.

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

Contributor

hvanhovell commented Apr 20, 2018

Merging to master. Thanks!

asfgit closed this in

074a7f9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment