HPCC-32856 CKeyCursor should link parent nodes as well as current node

More changes following review/testing/discussion Signed-off-by: Richard Chapman <[email protected]>
hpcc-systems · Nov 27, 2024 · 501c735 · 501c735
1 parent 0fb7331
commit 501c735
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/system/jhtree/jhtree.cpp b/system/jhtree/jhtree.cpp
@@ -1880,6 +1880,12 @@ bool CKeyCursor::_next(IContextLogger *ctx)
                             }
                             parents[depth].clear();
                         }
+                        else
+                        {
+                            // We can't tell which parent node info is correct. Clear them all.
+                            clearParentNodes();
+                            break;
+                        }
                     }
                     return node->isKeyAt(0);
                 }
@@ -1996,7 +2002,7 @@ bool CKeyCursor::_gtEqual(IContextLogger *ctx)
                         int rc = startnode->compareValueAt(recordBuffer, numParentKeys-1);
                         if (rc <= 0)
                         {
-                            lwm = parentNodeKey+1;
+                            lwm = parentNodeKey;
                             node.set(startnode);
                             break;
                         }
@@ -2098,7 +2104,7 @@ bool CKeyCursor::_ltEqual(IContextLogger *ctx)
                         int rc = startnode->compareValueAt(recordBuffer, numParentKeys-1);
                         if (rc < 0)
                         {
-                            lwm = parentNodeKey+1;
+                            lwm = parentNodeKey;
                             node.set(startnode);
                             break;
                         }