[ 
https://issues.apache.org/jira/browse/DERBY-4670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13039390#comment-13039390
 ] 

Myrna van Lunteren commented on DERBY-4670:
-------------------------------------------

I think we should consider closing this one as won't fix, because I don't think 
the class can cause any harm in the insane build, as there's nothing calling it.
The calls are from org.apache.derby.shared.common.sanity.AssertFailure via 
reflection. AssertFailure is only included in the sane builds.

ThreadDump only gets into the builds because it's listed in 
tools/jar/extraDBMSclasses.properties and tools/jar/dnc.properties, and there's 
no way to separate those into sane and insane.

Opinions?





> ThreadDump class included in insane jars
> ----------------------------------------
>
>                 Key: DERBY-4670
>                 URL: https://issues.apache.org/jira/browse/DERBY-4670
>             Project: Derby
>          Issue Type: Bug
>          Components: Build tools
>    Affects Versions: 10.6.1.0
>            Reporter: Knut Anders Hatlen
>
> The ThreadDump class in org.apache.derby.shared.common.sanity is included in 
> the insane versions of derby.jar and derbyclient.jar. The sanity classes 
> should only be included in debug builds.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to