[
https://issues.apache.org/jira/browse/DERBY-3701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12603413#action_12603413
]
Knut Anders Hatlen commented on DERBY-3701:
-------------------------------------------
Is the exists check necessary in the code below? In Sun's class library at
least, File.mkdirs() first calls File.exists() and does nothing if the
directory exists. Since mkdirs() doesn't throw IOException, I believe it should
be safe in other JVMs as well.
+ if (!PrivilegedFileOps.exists(traceDirectory))
+ {
+ PrivilegedFileOps.mkdirs(traceDirectory);
+ }
mkdirs()'s javadoc has an invalid HTML tag (<code> true </true>). Fixed in
revision 664524.
The indentation in Session.java seems to be off. Probably a wrong tab size
setting in your IDE. Fixed in revision 664530.
> java.lang.Exception: DRDA_UnableToAccept.S:Unable to accept connections and
> client hang if tracing is turned on but traceDirectory does not exist
> -------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: DERBY-3701
> URL: https://issues.apache.org/jira/browse/DERBY-3701
> Project: Derby
> Issue Type: Bug
> Components: Network Server
> Affects Versions: 10.3.2.1, 10.3.3.0, 10.4.1.3, 10.5.0.0
> Reporter: Kathey Marsden
> Assignee: Kathey Marsden
> Attachments: derby-3701_diff.txt, derby-3701_try1_diff.txt,
> DerbyIssue.java, releaseNote.html
>
>
> Attempting to connect to network server if derby.drda.traceAll is set to
> true and derby.drda.traceDirectory is set to a non-existent directory causes
> the following exception on the console when the client attempts to connect
> and causes the client to hang.
> java.lang.Exception: DRDA_UnableToAccept.S:Unable to accept connections.
> at
> org.apache.derby.impl.drda.NetworkServerControlImpl.consolePropertyMessageWork(NetworkServerControlImpl.java:
> 3172)
> at
> org.apache.derby.impl.drda.NetworkServerControlImpl.consolePropertyMessage(NetworkServerControlImpl.java:1829
> )
> at org.apache.derby.impl.drda.ClientThread.run(ClientThread.java:116)
> See attached program DerbyIssue.java for reproducible test case.
> This is a regression. It did not occur with 10.3.1.4.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
