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

We should add back concurrency ... #10

Open
bnlawrence opened this issue Sep 29, 2022 · 1 comment
Open

We should add back concurrency ... #10

bnlawrence opened this issue Sep 29, 2022 · 1 comment
Assignees
@bnlawrence
Labels
excalibur Needs discussion by the excalibur team
Milestone
Post-Prototype

Comments

@bnlawrence
Copy link
Collaborator

... when we get to actually doing performance testing. This version currently loops over chunks rather than doing them in parallel. Zarr has options to do both, so we could lift the relevant code (from here).
@bnlawrence
Copy link
Collaborator Author

Zarr currently does this itself, but it might be better to send a list of chunk requirements and let the storage do it, a bit more like the DAP protocol (or the DAP protocol itself, suggestion from Antonio via David)

@bnlawrence bnlawrence added the excalibur Needs discussion by the excalibur team label Oct 3, 2022
@bnlawrence bnlawrence self-assigned this Oct 27, 2022
@bnlawrence bnlawrence added this to the Post-Prototype milestone Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bnlawrence bnlawrence

Labels
excalibur Needs discussion by the excalibur team
Projects
None yet
Milestone
Post-Prototype
Development

No branches or pull requests
1 participant
@bnlawrence