Skip to content

Commit

Permalink
Fix the missing include_subsume flag so that subsumed tuples does n…
Browse files Browse the repository at this point in the history
…ot show up for rules (#474)

To see how this bug can cause failures, comment out the lines where `from_column_index` is used (which is hiding the bug) and use `LazyTrie::default` instead and run the test.
  • Loading branch information
yihozhang authored Dec 4, 2024
1 parent 86cb822 commit 246b195
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/gj.rs
Original file line number Diff line number Diff line change
Expand Up @@ -959,7 +959,7 @@ impl<'a> TrieAccess<'a> {
if idxs.is_empty() {
let rows = self
.function
.iter_timestamp_range(&self.timestamp_range, true);
.iter_timestamp_range(&self.timestamp_range, self.include_subsumed);
if self.column < arity {
for (i, tup, out) in rows {
insert(i, tup, out, tup[self.column])
Expand Down

0 comments on commit 246b195

Please sign in to comment.