Skip to content

Conversation

@andrewor14
Copy link
Contributor

It did not handle null keys very gracefully before.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16298/

@aarondav
Copy link
Contributor

aarondav commented Jul 3, 2014

LGTM, merging into master and branch-1.0.

@asfgit asfgit closed this in c480537 Jul 3, 2014
asfgit pushed a commit that referenced this pull request Jul 3, 2014
It did not handle null keys very gracefully before.

Author: Andrew Or <[email protected]>

Closes #1288 from andrewor14/fix-external and squashes the following commits:

312b8d8 [Andrew Or] Abstract key hash code
ed5adf9 [Andrew Or] Fix NPE for ExternalAppendOnlyMap

(cherry picked from commit c480537)
Signed-off-by: Aaron Davidson <[email protected]>
@aarondav
Copy link
Contributor

aarondav commented Jul 3, 2014

This fixes SPARK-2349.

@andrewor14
Copy link
Contributor Author

Thanks Aaron, I have updated the title.

@andrewor14 andrewor14 changed the title [SPARK] Fix NPE for ExternalAppendOnlyMap [SPARK-2349] Fix NPE for ExternalAppendOnlyMap Jul 3, 2014
@andrewor14 andrewor14 deleted the fix-external branch July 8, 2014 21:31
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
It did not handle null keys very gracefully before.

Author: Andrew Or <[email protected]>

Closes apache#1288 from andrewor14/fix-external and squashes the following commits:

312b8d8 [Andrew Or] Abstract key hash code
ed5adf9 [Andrew Or] Fix NPE for ExternalAppendOnlyMap
wangyum pushed a commit that referenced this pull request May 26, 2023
…pleUDF (#1288)

### What changes were proposed in this pull request?
- As a subtask of [SPARK-42050](https://issues.apache.org/jira/browse/SPARK-42050), this PR adds Codegen Support for HiveSimpleUDF
- Extract a`HiveUDFEvaluatorBase` class for the common behaviors of HiveSimpleUDFEvaluator & HiveGenericUDFEvaluator.

### Why are the changes needed?
- Improve codegen coverage and performance.
- Following #39949. Make the code more concise.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Add new UT.
Pass GA.

Closes #40397 from panbingkun/refactor_HiveSimpleUDF.

Authored-by: panbingkun <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
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