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 "Knut Anders Hatlen (JIRA)" <ji...@apache.org> on 2009/03/09 17:30:50 UTC

[jira] Created: (DERBY-4086) BTreeScan.isCurrentPositionDeleted() should return true if the row has been purged

BTreeScan.isCurrentPositionDeleted() should return true if the row has been purged
----------------------------------------------------------------------------------

                 Key: DERBY-4086
                 URL: https://issues.apache.org/jira/browse/DERBY-4086
             Project: Derby
          Issue Type: Bug
          Components: Store
    Affects Versions: 10.4.2.0
            Reporter: Knut Anders Hatlen
            Priority: Minor


Currently, BTreeScan.isCurrentPositionDeleted() returns false if the current position has been purged (that is, if reposition(scan_position, false) returns false). Since a row must be deleted in order to be purged, I believe it is more correct if isCurrentPositionDeleted() returns true in this case.

(I'm not sure if it is possible to exercise this code path in the current code. It is not exercised by any of the existing regression tests. Its only caller, TableScanResultSet.getCurrentRow(), also calls doesCurrentPositionQualify() which correctly returns false in this case, and it therefore skips the row and hides this problem. So even if the code can be exercised, it shouldn't cause any observable problems at this time.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (DERBY-4086) BTreeScan.isCurrentPositionDeleted() should return true if the row has been purged

Posted by "Rick Hillegas (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/DERBY-4086?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rick Hillegas updated DERBY-4086:
---------------------------------

    Urgency: Normal

Triaged July 2, 2009: Assigned normal urgency.

> BTreeScan.isCurrentPositionDeleted() should return true if the row has been purged
> ----------------------------------------------------------------------------------
>
>                 Key: DERBY-4086
>                 URL: https://issues.apache.org/jira/browse/DERBY-4086
>             Project: Derby
>          Issue Type: Bug
>          Components: Store
>    Affects Versions: 10.4.2.0
>            Reporter: Knut Anders Hatlen
>            Priority: Minor
>
> Currently, BTreeScan.isCurrentPositionDeleted() returns false if the current position has been purged (that is, if reposition(scan_position, false) returns false). Since a row must be deleted in order to be purged, I believe it is more correct if isCurrentPositionDeleted() returns true in this case.
> (I'm not sure if it is possible to exercise this code path in the current code. It is not exercised by any of the existing regression tests. Its only caller, TableScanResultSet.getCurrentRow(), also calls doesCurrentPositionQualify() which correctly returns false in this case, and it therefore skips the row and hides this problem. So even if the code can be exercised, it shouldn't cause any observable problems at this time.)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.