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.

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

Wrong argument is being passed to constructor of custom data type #113

Merged
merged 1 commit into from Apr 3, 2017

Conversation

Projects
None yet
2 participants
@rayokota
Contributor

rayokota commented Mar 31, 2017

What changes were proposed in this pull request?

To fix the argument passed to the constructor of a custom data type

How was this patch tested?

All existing unit tests still pass

@weiqingy weiqingy merged commit 6574ef3 into hortonworks-spark:master Apr 3, 2017

weiqingy added a commit that referenced this pull request Apr 3, 2017

Wrong argument is being passed to constructor of custom data type (#113)
## What changes were proposed in this pull request?

To fix the argument passed to the constructor of a custom data type

## How was this patch tested?

All existing unit tests still pass

weiqingy added a commit that referenced this pull request Apr 3, 2017

Wrong argument is being passed to constructor of custom data type (#113)
## What changes were proposed in this pull request?

To fix the argument passed to the constructor of a custom data type

## How was this patch tested?

All existing unit tests still pass

weiqingy added a commit that referenced this pull request Apr 3, 2017

Wrong argument is being passed to constructor of custom data type (#113)
## What changes were proposed in this pull request?

To fix the argument passed to the constructor of a custom data type

## How was this patch tested?

All existing unit tests still pass
@weiqingy

This comment has been minimized.

Show comment
Hide comment
@weiqingy

weiqingy Apr 3, 2017

Contributor

Thanks @rayokota . Merged to master/branch-1.6/branch-2.0/branch-2.1.

Contributor

weiqingy commented Apr 3, 2017

Thanks @rayokota . Merged to master/branch-1.6/branch-2.0/branch-2.1.

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