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.

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

Remove all non-standard CommandDests #1819

Closed
3 tasks done
derekjn opened this issue Jul 16, 2017 · 3 comments
Closed
3 tasks done

Remove all non-standard CommandDests #1819

derekjn opened this issue Jul 16, 2017 · 3 comments
Assignees
@derekjn
Milestone
0.9.8

Comments

@derekjn
Copy link
Contributor

derekjn commented Jul 16, 2017 •
edited
Loading

  • DestTupleTable
  • DestCombiner
  • DestTransform
@derekjn derekjn added this to the 0.9.8 milestone Jul 16, 2017
@derekjn derekjn self-assigned this Jul 16, 2017
@derekjn
Copy link
Contributor Author

derekjn commented Aug 8, 2017 •
edited
Loading

Each of these can probably be replaced fairly cleanly by using DestTuplestore and just writing those tuples out from the tuplestore to ZMQ as they currently are in the current CommandDests.

@derekjn
Copy link
Contributor Author

derekjn commented Aug 11, 2017 •
edited
Loading

@usmanm While this obviously requires a bit of extra iteration since output rows aren't written directly to the receiver, after some initial work it seems like this model will actually help quite a bit with #853.

@usmanm
Copy link
Collaborator

usmanm commented Aug 11, 2017

Yes, that's going to be a pretty killer feature. I think first step is going to be ordered stream processing. Which we can add as another "level", since it'll slow things down a bit.

@derekjn derekjn closed this as completed Aug 11, 2017
derekjn added a commit that referenced this issue Aug 11, 2017
@derekjn
#1819 Standardize use of DestReceivers
6e21c9c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derekjn derekjn

Labels
None yet
Projects
None yet
Milestone
0.9.8
Development

No branches or pull requests
2 participants
@derekjn @usmanm