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

Make sure to cleanup goroutine generated by the namespace manager and the parser #181

Merged
merged 2 commits into from
Oct 14, 2021

Conversation

josephschorr
Copy link
Member

@josephschorr josephschorr commented Oct 8, 2021

Uses dgraph-io/ristretto#286, imported by branch

@github-actions github-actions bot added area/api v0 Affects the v0 API area/dependencies Affects dependencies area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) labels Oct 8, 2021
@josephschorr josephschorr force-pushed the further-cleanup branch 2 times, most recently from 5727812 to c03ff58 Compare October 14, 2021 18:02
@josephschorr josephschorr marked this pull request as ready for review October 14, 2021 18:10
@jzelinskie jzelinskie added area/perf Affects performance or scalability area/schema Affects the Schema Language kind/bug Something is broken or regressed labels Oct 14, 2021
… the parser

Depends upon dgraph-io/ristretto#286

Signed-off-by: Joseph Schorr <josephschorr@users.noreply.github.com>
…etto#286 is merged

Signed-off-by: Joseph Schorr <josephschorr@users.noreply.github.com>
@jzelinskie jzelinskie merged commit d926ca4 into main Oct 14, 2021
@jzelinskie jzelinskie deleted the further-cleanup branch October 14, 2021 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api v0 Affects the v0 API area/dependencies Affects dependencies area/perf Affects performance or scalability area/schema Affects the Schema Language area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) kind/bug Something is broken or regressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants