Menu ▾ ▴

#350 'delete' seems to have issues with explicit index values

v1.1.x
fixed
Erik Hänel
critical (96)
v1.1.1
Bug Fix
2018-11-16
2018-06-09
Erik Hänel
No

The command delete is probably not able to handle explicit index values correctly, e.g. delete cache(:,col+1:) -ignore is not working.

Analysis:
The command delete itself is able to handle explicit index values. It is however possible that it doesn't work how it should if it is used within a loop. The approach is to ensure that the bytecode within a loop is used reasonably and that there are no side-effects if the loop mode is deactivated temporary.

Implementation:
The implementation was done as proposed by the analysis. Implementation tests indicated multiple optimization potentials, which were included into the implementation.

Documentation:
The implementation was added to the changes log. The added functions were commented in detail.

Tests:
This fix was added to the automatic SW tests. No deviations detected.

Discussion

  • Erik Hänel

    Erik Hänel - 2018-06-10
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2018-06-28
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,9 @@
 The command `delete` is probably not able to handle explicit index values correctly, e.g. `delete cache(:,col+1:) -ignore` is not working.
+
+**Analysis:**
+
+**Implementation:**
+
+**Documentation:**
+
+**Tests:**
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2018-07-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -1,6 +1,7 @@
 The command `delete` is probably not able to handle explicit index values correctly, e.g. `delete cache(:,col+1:) -ignore` is not working.

 **Analysis:**
+The command `delete` itself is able to handle explicit index values. It is however possible that it doesn't work how it should if it is used within a loop. The approach is to ensure that the bytecode within a loop is used reasonably and that there are no side-effects if the loop mode is deactivated temporary.

 **Implementation:**
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2018-07-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -4,7 +4,9 @@
 The command `delete` itself is able to handle explicit index values. It is however possible that it doesn't work how it should if it is used within a loop. The approach is to ensure that the bytecode within a loop is used reasonably and that there are no side-effects if the loop mode is deactivated temporary.

 **Implementation:**
+The implementation was done as proposed by the analysis. Implementation tests indicated multiple optimization potentials, which were included into the implementation.

 **Documentation:**
+The implementation was added to the changes log. The added functions were commented in detail.

 **Tests:**
    • status: implementing --> testing
     

  • Erik Hänel

    Erik Hänel - 2018-11-16
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -10,3 +10,4 @@
 The implementation was added to the changes log. The added functions were commented in detail.

 **Tests:**
+This fix was added to the automatic SW tests. No deviations detected.
    • status: testing --> fixed
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB