Skip to content

release/v1.2 - remove cache and compression from indexing#6325

Merged
NamanJain8 merged 4 commits intorelease/v1.2from
naman/remove-cache
Nov 20, 2020
Merged

release/v1.2 - remove cache and compression from indexing#6325
NamanJain8 merged 4 commits intorelease/v1.2from
naman/remove-cache

Conversation

@NamanJain8
Copy link
Copy Markdown
Contributor

@NamanJain8 NamanJain8 commented Aug 31, 2020

This PR removes the cache and compression from indexing which is not present in master and other release branches but is somehow present in v1.2.
This change was made to master through Use encryption in temp badger, fix compilation on 32bit commit, but was not cherry-picked to v1.2.


This change is Reviewable

Docs Preview: Dgraph Preview

Copy link
Copy Markdown
Contributor

@jarifibrahim jarifibrahim left a comment

Choose a reason for hiding this comment

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

:lgtm: Please merge this PR only after the current release is done. We don't need this in the current release.
Also, list the commit that removed cache from indexing in master in the commit message of this PR.

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @jarifibrahim, @manishrjain, and @martinmr)

@NamanJain8 NamanJain8 merged commit 3f03832 into release/v1.2 Nov 20, 2020
@NamanJain8 NamanJain8 deleted the naman/remove-cache branch November 20, 2020 11:28
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.

2 participants