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

Add chunked processing back into v2 client #7989

Merged
merged 1 commit into from
Feb 13, 2017

Conversation

jgeiger
Copy link

@jgeiger jgeiger commented Feb 10, 2017

  • Moving the to v2 client removed this functionality. This copies
    code back into the client. The tests were also added back into
    the test suite.
Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

Fixes #7977

@jwilder jwilder requested a review from jsternberg February 10, 2017 17:19
@jwilder jwilder added this to the 1.3.0 milestone Feb 10, 2017
@jsternberg
Copy link
Contributor

Can you rebase this and then I'll merge it?

@jsternberg jsternberg self-assigned this Feb 13, 2017
- Moving the to v2 client removed this functionality. This copies
  code back into the client. The tests were also added back into
  the test suite.
@jgeiger
Copy link
Author

jgeiger commented Feb 13, 2017

Done. Thanks.

@jsternberg
Copy link
Contributor

The build error looks unrelated to I'm going to merge this.

@jsternberg jsternberg merged commit 19f331a into influxdata:master Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants