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.

Sign up for GitHub

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

Jump to bottom

Remove unused stream #7683

Closed
wants to merge 1 commit into from
Closed

Remove unused stream #7683

wants to merge 1 commit into from

Conversation

bleskes
Copy link
Contributor

@bleskes bleskes commented Sep 11, 2014

No description provided.

@bleskes bleskes added the review label Sep 11, 2014
@s1monw
Copy link
Contributor

s1monw commented Sep 11, 2014

good catch!! LGTM

@dakrone
Copy link
Member

dakrone commented Sep 11, 2014

LGTM too.

@bleskes bleskes closed this in 6849fd0 Sep 11, 2014
bleskes added a commit that referenced this pull request Sep 11, 2014
@bleskes
Translog: remove unused stream
47b25bc 
Closes #7683
@bleskes bleskes deleted the translog_always_close_stream branch September 11, 2014 08:07
@jpountz jpountz removed the review label Oct 21, 2014
@clintongormley clintongormley changed the title Translog: remove unused stream Remove unused stream Jun 7, 2015
@clintongormley clintongormley added :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. and removed :Translog :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
:Distributed/Engine Anything around managing Lucene and the Translog in an open shard. >enhancement v1.4.0.Beta1 v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@bleskes @s1monw @dakrone @clintongormley @jpountz