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

Eliminate deep copying of batch points #281

Closed
sparrc opened this issue Oct 16, 2015 · 0 comments
Closed

Eliminate deep copying of batch points #281

sparrc opened this issue Oct 16, 2015 · 0 comments

Comments

@sparrc
Copy link
Contributor

sparrc commented Oct 16, 2015

Multiple output sinks each need to be passed the same BatchPoints, which requires us to deepcopy the entire batch for each output, See #250

If we make a lockable wrapper, we can eliminate the deep copying.
#280 should be done before this is worked on.
sparrc added a commit that referenced this issue Oct 16, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
291fc06 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 16, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
c4f621d 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 16, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
38a9a86 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 16, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
a8aaaa9 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 16, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
727dcbb 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 17, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
bca16b7 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
95680e1 
Fixes #280
Fixes #281
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
76e1005 
Fixes #280
Fixes #281
@sparrc sparrc mentioned this issue Oct 20, 2015
Merged
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
313632c 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
d1cac0f 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
c39f75f 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
daf11d5 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
200cd2e 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
cf0a1db 
Fixes #280
Fixes #281
Fixes #289
@sparrc sparrc changed the title Create a lockable wrapper of client.BatchPoints Eliminate deep copying of batch points Oct 20, 2015
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
a64c037 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
7410e2f 
Fixes #280
Fixes #281
Fixes #289
sparrc added a commit that referenced this issue Oct 20, 2015
@sparrc
Utilizing new client and overhauling Accumulator interface
c26ce9c 
Fixes #280
Fixes #281
Fixes #289
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sparrc