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

Unreleased Resource: Streams #1336

Closed
QiAnXinCodeSafe opened this issue Sep 14, 2020 · 2 comments · Fixed by #1350
Closed

Unreleased Resource: Streams #1336

QiAnXinCodeSafe opened this issue Sep 14, 2020 · 2 comments · Fixed by #1350

Comments

@QiAnXinCodeSafe
Copy link

The program can potentially fail to release a system resource.

@abracadv8
Copy link
Contributor

I can submit a quick pr - is there a corporate CLA?

I'll just wrap it in a finally similar to how it's done in other sections of the code like here: https://github.com/Netflix/eureka/blob/master/eureka-client-jersey2/src/main/java/com/netflix/discovery/shared/transport/jersey2/EurekaJersey2ClientImpl.java#L313

@troshko111
Copy link
Contributor

troshko111 commented Oct 6, 2020

Nope, no CLA, PRs welcomed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants