You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Rick Hillegas (JIRA)" <de...@db.apache.org> on 2006/09/19 17:35:22 UTC
[jira] Updated: (DERBY-813) Clean up coding error in
searchClauseTransitiveClosure method in PredicateList.java
[ http://issues.apache.org/jira/browse/DERBY-813?page=all ]
Rick Hillegas updated DERBY-813:
--------------------------------
Fix Version/s: 10.2.2.0
(was: 10.2.1.0)
Moving to 10.2.2.0.
> Clean up coding error in searchClauseTransitiveClosure method in PredicateList.java
> -----------------------------------------------------------------------------------
>
> Key: DERBY-813
> URL: http://issues.apache.org/jira/browse/DERBY-813
> Project: Derby
> Issue Type: Sub-task
> Components: SQL
> Affects Versions: 10.0.2.0
> Environment: generic
> Reporter: Satheesh Bandaram
> Priority: Minor
> Fix For: 10.2.2.0
>
>
> I am separating this part of the bug from DERBY-808. This part of the fix can go into 10.2 release, while I would like to submit the first part of DERBY-808 to both 10.1 and trunk branches.
> Coding error in PredicateList.searchClauseTransitiveClosure. Also note, when the operands are switched, the type of the operation needs to change. For example, <ConstantNode> <LessThan> <ColumnReference> should change to <ColumnReference> <GreaterThan> <Constant>. Current code doesn's seem to do that either.
> if (left instanceof ColumnReference && right instanceof ConstantNode)
> {
> searchClauses.addElement(predicate);
> }
> else if (right instanceof ConstantNode && left instanceof ColumnReference)
> {
> // put the ColumnReference on the left to simplify things
> bcon.swapOperands();
> searchClauses.addElement(predicate);
> }
> The second part of the check is incorrect. It should instead be like:
> else if (left instanceof ConstantNode && right instanceof ColumnReference)
> and while handling ParameterNodes. Also the swaping logic is not complete.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira