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.

Sign up for GitHub

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

Jump to bottom

Add several backports to 1.6 #10003

Merged
merged 10 commits into from
Jun 27, 2018
Merged

Add several backports to 1.6 #10003

merged 10 commits into from
Jun 27, 2018

Conversation

jacobmarble
Copy link
Member

Fixes #10000

@jacobmarble jacobmarble self-assigned this Jun 26, 2018
@ghost ghost added the review label Jun 26, 2018
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

3 similar comments
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@jacobmarble jacobmarble force-pushed the jgm-1.6-backports branch 2 times, most recently from 8a69d5c to 1a75b94 Compare June 27, 2018 00:06
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

1 similar comment
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@jacobmarble jacobmarble force-pushed the jgm-1.6-backports branch 2 times, most recently from 241fb42 to 40b5453 Compare June 27, 2018 00:08
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

1 similar comment
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@e-dard
Copy link
Contributor

e-dard commented Jun 27, 2018

@jacobmarble there are bunch of other dependencies that need to go into Godeps I think. Mainly from klauspost. I think they were missed off of #9856. You can cherry pick #9983 in I think to sort it.

@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

e-dard and others added 8 commits June 27, 2018 08:36
@e-dard @jacobmarble
Merge pull request #9967 from influxdata/pd-prometheus-to-new-storage
2a52243 
Use storage package for Prometheus remote read
@benbjohnson @jacobmarble
Merge pull request #9969 from influxdata/bj-cli-retention-policy
841e9d2 
Add retention policy flag to CLI queries.
@jacobmarble
Merge pull request #9972 from influxdata/jgm-snapshot
d3254b5 
TSM: Fix ShouldCompactCache without WAL
@e-dard @jacobmarble
Merge pull request #9856 from pliu/backup_restore_parallel_compression
c4b0703 
Parallelize compression/decompression during backup and restore
@dgnorton @jacobmarble
Merge pull request #9970 from influxdata/dn-show-tag-keys-perf
aba6e8c 
fix SHOW TAG KEYS perfomance regression
@e-dard @jacobmarble
Merge pull request #9983 from influxdata/er-deps
a4c443d 
Fix Godeps
@benbjohnson @jacobmarble
Merge pull request #9982 from influxdata/bj-tsi-wal-flush-sync
d7298e3 
Flush/sync TSI1 WAL
@zeebo @jacobmarble
Merge pull request #10001 from influxdata/jmw-fix-tsm1-sync
8bd0895 
tsm1: ensure all written tsm files are fsynced
@jacobmarble
Copy link
Member Author

dep ensure makes no change to Gopkg.lock.

@jacobmarble jacobmarble requested a review from e-dard June 27, 2018 15:38
@jacobmarble
Copy link
Member Author

@e-dard PTAL

e-dard
e-dard approved these changes Jun 27, 2018
View reviewed changes
Copy link
Contributor

@e-dard e-dard left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jacobmarble jacobmarble merged commit c596e0b into 1.6 Jun 27, 2018
@ghost ghost removed the review label Jun 27, 2018
@jacobmarble jacobmarble deleted the jgm-1.6-backports branch June 27, 2018 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@e-dard e-dard e-dard approved these changes

Assignees

@jacobmarble jacobmarble

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@jacobmarble @hercules-influx @e-dard @stuartcarnie @benbjohnson @dgnorton @zeebo