-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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 frequently used IO resources autoclosable #8181
Make frequently used IO resources autoclosable #8181
Conversation
Hmm the Kotlin changes also break my PR :( |
Yea, something untested got merged to master, dont worry about it. |
58b2f4b
to
9d1c3ad
Compare
cc @OpenAPITools/generator-core-team |
The kotlin samples have been updated in the master. Please merge the latest master into this branch when you've time. |
…to auto-closable-IO-resources
@wing328 done! |
…to auto-closable-IO-resources
…to auto-closable-IO-resources
…to auto-closable-IO-resources
…to auto-closable-IO-resources
@OpenAPITools/generator-core-team @wing328 |
@wing328 @jimschubert any plans to merge it? |
Let me try to review over the weekend. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and the result is good.
Thanks |
to reduce memory use and unlock them for use by operating system resource scheduler/allocator
Below you can see I run the same template manager test which opens and reads IO resources. I added extra line on debugger just to keep resources busy a bit longer for lsof to scan though them, this allows me to compare before/after results of my changes, as it would be much more difficult to automate this test inside maven, it's all manual.
List of files open by Linux checked with
Before changes:
Same test after changes:
Also reduced complexity of reading dart specific keywords from external file.
Autoclose input reader from ruby error stream
I do not know what the kotlin changes are, I promise ¯_(ツ)_/¯
Closes #1909 and fixes a few other unreported