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

Ensure adequate testing for lazy data merge and concatenate (3 man days) #2384

Closed
pp-mo opened this issue Feb 16, 2017 · 1 comment
Closed

Ensure adequate testing for lazy data merge and concatenate (3 man days) #2384

pp-mo opened this issue Feb 16, 2017 · 1 comment
Assignees
@djkirkham
Milestone
v2.0.0

Comments

@pp-mo
Copy link
Member

pp-mo commented Feb 16, 2017 •
edited
Loading

At present, tests/unit/concatenate/test_concatenate.TestConcatenateBiggus is skipped, but really needs to be replaced by a dask-aware version.
I couldn't find anything specifically testing lazy merging -- I guess there should be something.
@QuLogic QuLogic added this to the dask milestone Feb 16, 2017
@lbdreyer lbdreyer changed the title Ensure adequate testing for lazy data merge and concatenate. Ensure adequate testing for lazy data merge and concatenate (3 man days) Apr 10, 2017
@bjlittle bjlittle added the dask label Apr 20, 2017
@bjlittle bjlittle mentioned this issue Apr 27, 2017
Closed
5 tasks
@djkirkham
Copy link
Contributor

djkirkham commented May 2, 2017 •
edited
Loading

Extra tests have been added by #2492. I'm happy that they provide enought coverage.

@pp-mo @bjlittle @dkillick please comment.

@bjlittle bjlittle closed this as completed May 3, 2017
@QuLogic QuLogic modified the milestones: dask, v2.0 Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@djkirkham djkirkham

Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

No branches or pull requests
4 participants
@QuLogic @bjlittle @pp-mo @djkirkham