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

Unify search context cleanup #7643

Merged
merged 1 commit into from Sep 8, 2014

Conversation

Projects
None yet
3 participants
@s1monw
Copy link
Contributor

commented Sep 8, 2014

Today there are two different ways to cleanup search contexts which can
potentially lead to double releasing of a context. This commit unifies
the methods and prevents double closing.

Closes #7625

@s1monw s1monw added the review label Sep 8, 2014

@martijnvg

This comment has been minimized.

Copy link
Member

commented Sep 8, 2014

LGTM

@martijnvg martijnvg removed the review label Sep 8, 2014

[CORE] Unify search context cleanup
Today there are two different ways to cleanup search contexts which can
potentially lead to double releasing of a context. This commit unifies
the methods and prevents double closing.

Closes #7625

@s1monw s1monw force-pushed the s1monw:issues/7625 branch to 72c4cb5 Sep 8, 2014

@s1monw s1monw merged commit 72c4cb5 into elastic:master Sep 8, 2014

@s1monw s1monw deleted the s1monw:issues/7625 branch Sep 8, 2014

@clintongormley clintongormley changed the title [Internal] Unify search context cleanup Internal: Unify search context cleanup Sep 11, 2014

@clintongormley clintongormley changed the title Internal: Unify search context cleanup Unify search context cleanup Jun 7, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.