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

Always append shard path in diags #2431

Merged
merged 1 commit into from
Apr 26, 2015
Merged

Always append shard path in diags #2431

merged 1 commit into from
Apr 26, 2015

Conversation

otoolep
Copy link
Contributor

@otoolep otoolep commented Apr 25, 2015

This code is clearer -- simply append an empty path if the shard is not
local.

Fixes issue #2430

@otoolep
Copy link
Contributor Author

otoolep commented Apr 25, 2015

Before this change the column path was added to the diags columns if the shard was local, otherwise it was not. That code had some bugs, so it's much simpler to just add the path column for all shards and simply set the path value to "" for non-local shards.

@marcosnils
Copy link
Contributor

@otoolep PR is clean and better. 👍 . I didn't want to change too much as I still don't know how internals work.

@otoolep
Copy link
Contributor Author

otoolep commented Apr 25, 2015

@mark-rushakoff

@toddboom
Copy link
Contributor

+1

This code is clearer -- simply append an empty path if the shard is not
local.

Fixes issue #2430
@otoolep otoolep force-pushed the diags_path_issues branch from b0beda8 to 257aa4d Compare April 26, 2015 18:01
otoolep added a commit that referenced this pull request Apr 26, 2015
Always append shard path in diags
@otoolep otoolep merged commit 86b9ec5 into master Apr 26, 2015
@otoolep otoolep deleted the diags_path_issues branch April 26, 2015 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants