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

Remove deprecated JSON write path #6025

Merged
merged 2 commits into from
Mar 16, 2016
Merged

Remove deprecated JSON write path #6025

merged 2 commits into from
Mar 16, 2016

Conversation

gunnaraasen
Copy link
Member

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

Removes the JSON write path that was deprecated in v0.11.

Also checks that writes include a valid database before decoding points and removes the check for the precision URL parameter, which will always default to nanoseconds if not provided.

@e-dard
Copy link
Contributor

e-dard commented Mar 16, 2016

LGTM 👍

@jwilder jwilder added this to the 0.12.0 milestone Mar 16, 2016
@jwilder
Copy link
Contributor

jwilder commented Mar 16, 2016

👍

@toddboom
Copy link
Contributor

+:100:

gunnaraasen added a commit that referenced this pull request Mar 16, 2016
Remove deprecated JSON write path
@gunnaraasen gunnaraasen merged commit c6ff258 into master Mar 16, 2016
@gunnaraasen gunnaraasen deleted the ga-remove-json branch March 16, 2016 21:17
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.

4 participants