scossu/volksdata_lua
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases 2 Packages Wiki Activity Actions

graph:remove() does not remove all triples in mdb store #3

New issue
Closed
opened 2026-01-16 22:22:41 +00:00 by scossu · 4 comments
scossu commented 2026-01-16 22:22:41 +00:00
Owner
Copy link

Given a graph gr in MDB store with 48 triples:

>  #gr
48
>  gr:remove()
48
>  #gr
6
>  gr:remove()
6
>  #gr
0

It takes several attempts (some times more than 2) to clear all triples. Note that the remove() function returns the number of triples that should have been removed.

This does not seem to affect the hash table store.

The number of triples left behind is consistent across repetitions. It changes depending on the number of triples present.

Also, this only happens if all the arguments are not passed or are nil. Adding any term that is common to all triples results in a full removal.

Given a graph `gr` in MDB store with 48 triples: ```lua > #gr 48 > gr:remove() 48 > #gr 6 > gr:remove() 6 > #gr 0 ``` It takes several attempts (some times more than 2) to clear all triples. Note that the `remove()` function returns the number of triples that should have been removed. This does not seem to affect the hash table store. The number of triples left behind is consistent across repetitions. It changes depending on the number of triples present. Also, this only happens if all the arguments are not passed or are nil. Adding any term that is common to all triples results in a full removal.
scossu commented 2026-01-16 22:25:31 +00:00
Author
Owner
Copy link

https://git.knowledgetx.com/scossu/volksdata/src/branch/master/test/test_graph.c#L585 passes, which seems to confirm that the issue is not with the underlying C library.

https://git.knowledgetx.com/scossu/volksdata/src/branch/master/test/test_graph.c#L585 passes, which seems to confirm that the issue is not with the underlying C library.
scossu commented 2026-01-16 23:24:37 +00:00
Author
Owner
Copy link

A test generating the same number of random triples fails with different counts.

A test generating the same number of random triples fails with different counts.
scossu commented 2026-01-17 06:17:43 +00:00
Author
Owner
Copy link

Actually, a new test with randomized values revealed that this problem is also present in the C library. Moving diagnosis one level lower.

Actually, a new test with randomized values revealed that this problem is also present in the C library. Moving diagnosis one level lower.
scossu commented 2026-01-17 21:50:32 +00:00
Author
Owner
Copy link

Fixed in 7e424d1889.

Fixed in 7e424d1889.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
beta
v1.0beta
v1.0b4
v1.0b3
v1.0b2
v1.0b1
Labels
Clear labels   
p
critical

  
p
high

  
p
low

  
p
medium

  
s
blocked
Work cannot continue before resolving another issue. The blocking cause must be specified.

  
s
progress
Work is in progress.

  
s
testing
A resolution has been been developed in a feature branch and is being tested. Once tests pass, it can be merged to master.

  
s
wontfix
The issue cannot be fixed. Either it's a duplicate, or irrelevant, or out of scope. In case of a legitimate issue, a workaround should be provided.

  
t
bug

  
t
documentation

  
t
enhancement

  
t
feature

  
t
test

No labels
p
critical
p
high
p
low
p
medium
s
blocked
s
progress
s
testing
s
wontfix
t
bug
t
documentation
t
enhancement
t
feature
t
test
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
scossu/volksdata_lua#3
No description provided.