Skip to content
Closed
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
9a08db4
First steps towards prototype for UPDATE
Mar 15, 2023
4bfc02e
New version with a first proper test
Mar 17, 2023
aec6370
Triple location code is now correct and well tested
Mar 18, 2023
9680383
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Mar 18, 2023
606f412
Insert and delete work properly now
Mar 19, 2023
d2d99e0
Make clang-format happy
Mar 19, 2023
4017177
Try to make the more pedantic native build happy
Mar 19, 2023
b4a81df
The permutations now have access to the delta triples
Mar 21, 2023
4d22469
Better names and own class `LocatedTriple`
Mar 21, 2023
f9506a8
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Mar 21, 2023
04861ce
Refactor code and first try to merge delta triples
Mar 31, 2023
0dbd01e
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Mar 31, 2023
82911e7
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Apr 1, 2023
10c74f0
Method `mergeTriples` with good unit tests
Apr 1, 2023
fa49a33
Consider delta triples for index scans with two variables
Apr 2, 2023
15eaf48
Delta triples merged for both scan types now
Apr 3, 2023
38ca1fd
Delta triples now considered for all blocks
Apr 4, 2023
ad743bb
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Apr 5, 2023
7a36a9d
Resolve conflict that git silently merged in the last commit
Apr 6, 2023
b10ec84
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Apr 6, 2023
3afe571
Address bug pointed out by SonarCloud
Apr 8, 2023
2f4b105
Revert "Address bug pointed out by SonarCloud"
Apr 11, 2023
48578d3
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Apr 17, 2023
3906826
Merge branch 'master' into update-proof-of-concept
May 24, 2023
ad1163c
Make it compile again
Jun 8, 2023
ebd075c
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Jun 8, 2023
8cd5186
Merge remote-tracking branch 'origin/master' into update-proof-of-con…
Jun 8, 2023
b881717
Fix test with help of Johannes
Jun 9, 2023
e88c293
Latest submodules
Jun 9, 2023
8910f61
Some clean up before splitting off a first smaller PR
Jun 9, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address bug pointed out by SonarCloud
I am not convinced though that it was a bug.
  • Loading branch information
Hannah Bast committed Apr 8, 2023
commit 3afe571c8943c3b0532be0f239a87d0534fad73c
10 changes: 5 additions & 5 deletions src/index/CompressedRelation.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -367,13 +367,13 @@ void CompressedRelationReader::scan(
ad_utility::AllocatorWithLimit<Id> allocator{
ad_utility::makeAllocationMemoryLeftThreadsafeObject(
std::numeric_limits<size_t>::max())};
IdTable result(1, allocator);
result.resize(block.size());
std::unique_ptr<IdTable> result = std::make_unique<IdTable>(1, allocator);
result->resize(block.size());
for (size_t i = 0; i < block.size(); ++i) {
result(i, 0) = block(i, 1);
(*result)(i, 0) = block(i, 1);
}
return {std::make_unique<IdTable>(std::move(result)), rowIndexBegin,
rowIndexEnd, blockIndex, numInsAndDel};
return {std::move(result), rowIndexBegin, rowIndexEnd, blockIndex,
numInsAndDel};
};

// The first and the last block might be incomplete. We process them
Expand Down