You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2020/01/04 14:34:41 UTC

[GitHub] [incubator-dolphinscheduler] Jave-Chen opened a new issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.

Jave-Chen opened a new issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.
URL: https://github.com/apache/incubator-dolphinscheduler/issues/1714
 
 
   **Describe the bug**
   Connections, streams, files, and other classes that implement the Closeable interface or its super-interface, AutoCloseable, needs to be closed after use. Further, that close call must be made in a finally block otherwise an exception could keep the call from being made. Preferably, when class implements AutoCloseable, resource should be created using "try-with-resources" pattern and will be closed automatically.
   
   Failure to properly close resources will result in a resource leak which could bring first the application and then perhaps the box it's on to their knees.
   
   **File**
   dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/utils/FourLetterWordMain.java
   dolphinscheduler-server/src/main/java/org/apache/dolphinscheduler/server/worker/task/sql/SqlTask.java
   
   **Link**
   https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&issues=AW9Pvju3YzPBRjkobYuy&open=AW9Pvju3YzPBRjkobYuy
   
   https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&issues=AW9PvkJkYzPBRjkobY5H&open=AW9PvkJkYzPBRjkobY5H
   https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&issues=AW9PvkJkYzPBRjkobY5F&open=AW9PvkJkYzPBRjkobY5F
   https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&issues=AW9PvkJkYzPBRjkobY5G&open=AW9PvkJkYzPBRjkobY5G

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-dolphinscheduler] Technoboy- closed issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.

Posted by GitBox <gi...@apache.org>.
Technoboy- closed issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.
URL: https://github.com/apache/incubator-dolphinscheduler/issues/1714
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [incubator-dolphinscheduler] Technoboy- commented on issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on issue #1714: [BUG] Fix bug: Use try-with-resources or close this "Socket" in a "finally" clause.
URL: https://github.com/apache/incubator-dolphinscheduler/issues/1714#issuecomment-575587497
 
 
   #1716

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services