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.

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

kvs: minor cleanup & refactoring #1402

Merged
merged 2 commits into from Mar 30, 2018

Conversation

3 participants
@chu11
Copy link
Contributor

chu11 commented Mar 30, 2018

These are two patches pulled out of #1397. The primary fix from #1397 was dropped, but these remaining cleanups/minor refactor can still be applied.

chu11 added some commits Mar 27, 2018

modules/kvs: Initialize walk at new time
Move where the lookup walk() is initialized, so that it is initialized
just before beginning a walk().

@chu11 chu11 added this to To do in multi-user parallel jobs via automation Mar 30, 2018

@chu11 chu11 moved this from To do to In progress in multi-user parallel jobs Mar 30, 2018

@coveralls

This comment has been minimized.

Copy link

coveralls commented Mar 30, 2018

Coverage Status

Coverage increased (+0.06%) to 78.827% when pulling 6629a18 on chu11:issue1341-part3-try2 into 25f7d40 on flux-framework:master.

@garlick

This comment has been minimized.

Copy link
Member

garlick commented Mar 30, 2018

Thanks!

@garlick garlick merged commit a7402b5 into flux-framework:master Mar 30, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.06%) to 78.827%
Details

multi-user parallel jobs automation moved this from In progress to Done Mar 30, 2018

@grondo grondo referenced this pull request May 10, 2018

Closed

0.9.0 Release #1479

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.