Skip to content
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

[FEA] Add in Broadcast support for decimal values #1326

Closed
revans2 opened this issue Dec 8, 2020 · 1 comment · Fixed by #1475
Closed

[FEA] Add in Broadcast support for decimal values #1326

revans2 opened this issue Dec 8, 2020 · 1 comment · Fixed by #1475
Assignees
Labels
feature request New feature or request

Comments

@revans2
Copy link
Collaborator

revans2 commented Dec 8, 2020

Is your feature request related to a problem? Please describe.
To support other operations we need to be able to send decimal types from one task to another (Like join #1325)

The code should already support this, but we need to enable it and test.

@revans2 revans2 added feature request New feature or request ? - Needs Triage Need team to review and classify labels Dec 8, 2020
@revans2 revans2 mentioned this issue Dec 8, 2020
27 tasks
@revans2
Copy link
Collaborator Author

revans2 commented Dec 10, 2020

I didn't think about how to test this properly. Broadcast really needs Joins working, because broadcast is not used without joins right now. So I will do Shuffle separately and keep this just for Broadcast.

@revans2 revans2 changed the title [FEA] Add in Shuffle and Broadcast support for decimal values [FEA] Add in Broadcast support for decimal values Dec 10, 2020
@sameerz sameerz removed the ? - Needs Triage Need team to review and classify label Dec 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants