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

Lint tsdb and tsdb/engine package #5375

Merged
merged 1 commit into from
Feb 10, 2016
Merged

Lint tsdb and tsdb/engine package #5375

merged 1 commit into from
Feb 10, 2016

Conversation

nussjustin
Copy link
Contributor

  • CHANGELOG.md updated
  • Rebased/mergable
  • Tests pass
  • Sign CLA (if not already signed)

Updates #4098

@nussjustin
Copy link
Contributor Author

Rebased

@@ -109,6 +109,8 @@ func (b *PointBatcher) Start() {
}()
}

// Stop stopts the batching process. Stop waits for the batching routine
Copy link
Contributor

Choose a reason for hiding this comment

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

stopts

@e-dard
Copy link
Contributor

e-dard commented Feb 10, 2016

@nuss-justin can you sign the CLA? (I don't see you on there under nuss-justin).

@nussjustin
Copy link
Contributor Author

@e-dard I signed the CLA (again?)

@jwilder
Copy link
Contributor

jwilder commented Feb 10, 2016

I think #5196 might have removed a lot of code changed here.

@nussjustin
Copy link
Contributor Author

@jwilder Updated to current master :-)

@jwilder
Copy link
Contributor

jwilder commented Feb 10, 2016

👍 Thanks @nuss-justin

jwilder added a commit that referenced this pull request Feb 10, 2016
Lint tsdb and tsdb/engine package
@jwilder jwilder merged commit 8ebcfd9 into influxdata:master Feb 10, 2016
@jwilder jwilder added this to the 0.11.0 milestone Feb 10, 2016
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.

None yet

3 participants