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

update to latest GKL with compression related optimizations #4379

Merged
merged 1 commit into from
Feb 20, 2018

Conversation

pnvaidya
Copy link
Collaborator

@pnvaidya pnvaidya commented Feb 8, 2018

The earlier GKL release did not have all updates merged. Hence submitting another request to include the latest GKL.

@droazen
Copy link
Contributor

droazen commented Feb 8, 2018

Can you provide a summary of the optimizations included here @pnvaidya ?

@droazen droazen self-requested a review February 8, 2018 21:38
@droazen droazen self-assigned this Feb 8, 2018
@pnvaidya
Copy link
Collaborator Author

pnvaidya commented Feb 8, 2018

Its the same optimizations for level as we discussed before, no new optimizations. The previous merge did not include all the optimizations (error on my part)... hence I have to do a new PR.

@droazen droazen merged commit ecf3107 into master Feb 20, 2018
@droazen droazen deleted the pv_compression_update branch February 20, 2018 17:19
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.

2 participants