db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Myrna van Lunteren (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DERBY-6453) Remove dead code in InsertResultSet and flag skipCheckConstraints
Date Mon, 13 Jan 2014 20:04:52 GMT

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

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

In my automated nightly builds, I saw a javadoc warning resulting from removing the skipCheckConstraints
in DMLModStatementNode.java. I could not figure out how to make this error show up in my 'manual'
environments, but it seemed that the warning was correct, so I removed the line.

> Remove dead code in InsertResultSet and flag skipCheckConstraints
> -----------------------------------------------------------------
>
>                 Key: DERBY-6453
>                 URL: https://issues.apache.org/jira/browse/DERBY-6453
>             Project: Derby
>          Issue Type: Improvement
>          Components: SQL
>            Reporter: Dag H. Wanvik
>            Assignee: Dag H. Wanvik
>            Priority: Minor
>             Fix For: 10.11.0.0
>
>         Attachments: cleanup-misc-2.diff, cleanup-misc-2.stat, cleanup-misc-3.diff, cleanup-misc.diff,
cleanup-misc.stat
>
>
> I identified some unused code; a flag that is always false, and code for handling triggers
in bulk insert which is unused (by inspection and corroborated by JaCoCo results for our regression
tests): we disable bulk insert if there are triggers present, cf. 
> * DMLModStatementNode#getAllRelevantTriggers ->
>   adjustDeferredFlag(true)
> * InsertResultSet#bulkInsert:
>   bulkInsert = verifyBulkInsert()->
>       :
>       // bulk insert is disabled for deferred mode inserts
>       if (constants.deferred)
>       return false;



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message