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

SHOW SHARD GROUPS #4792

Merged
merged 1 commit into from
Nov 19, 2015
Merged

Conversation

mateuszdyminski
Copy link
Contributor

This commit fixes #4728

@otoolep
Copy link
Contributor

otoolep commented Nov 14, 2015

Thanks @mateuszdyminski -- have you signed the CLA?

@@ -2,6 +2,7 @@

### Features
- [#4790](https://github.com/influxdb/influxdb/pull/4790): Allow openTSDB point-level error logging to be disabled
- [#4728](https://github.com/influxdb/influxdb/pull/4728): SHOW SHARD GROUPS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add 'by @mateuszdyminski' to the end of the line. We like to include credits in the CHANGELOG.

@otoolep
Copy link
Contributor

otoolep commented Nov 14, 2015

Looks good @mateuszdyminski -- some small changes required.

@dgnorton -- can I get a +1 assuming requested changes are made?

@mateuszdyminski
Copy link
Contributor Author

@otoolep I signed the CLA.

All comments fixed.

@otoolep
Copy link
Contributor

otoolep commented Nov 15, 2015

CLA confirmed signed.

@dgnorton -- please review and merge if it looks good to you.

@otoolep
Copy link
Contributor

otoolep commented Nov 15, 2015

+1

1 similar comment
@corylanou
Copy link
Contributor

+1

otoolep added a commit that referenced this pull request Nov 19, 2015
@otoolep otoolep merged commit 68408a7 into influxdata:master Nov 19, 2015
@otoolep
Copy link
Contributor

otoolep commented Nov 19, 2015

Thanks @mateuszdyminski !

@mateuszdyminski
Copy link
Contributor Author

Thanks for help @otoolep. It was my first open-source contribution!

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.

SHOW SHARD GROUPS
3 participants