Skip to content
New issue

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

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

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants