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

docs: Update FAQ regarding ZSTD and CGO #1766

Merged

Conversation

BenjamenMeyer
Copy link

@BenjamenMeyer BenjamenMeyer commented Dec 23, 2021

  • the library used for ZSTD was replaced by a pure Golang version
    so CGO is no longer required. Update the FAQ to reflect this
  • link to the discussion regarding backwards compatibility
    so it's easy to find.

This change is Reviewable

- the library used for ZSTD was replaced by a pure Golang version
  so CGO is no longer required. Update the FAQ to reflect this
- link to the discussion regarding backwards compatibility
  so it's easy to find.
@NamanJain8
Copy link
Contributor

Thanks a lot @BenjamenMeyer !!

cc: @danielmai

@danielmai danielmai changed the title Enhancement: Update FAQ regarding ZSTD and CGO docs: Update FAQ regarding ZSTD and CGO Dec 23, 2021
@danielmai
Copy link
Contributor

Thanks for the PR, @BenjamenMeyer!

@danielmai danielmai merged commit f4039f0 into dgraph-io:master Dec 23, 2021
joshua-goldstein added a commit that referenced this pull request Feb 9, 2023
mangalaman93 pushed a commit that referenced this pull request Feb 11, 2023
mangalaman93 pushed a commit that referenced this pull request Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants