Tobias Jordan
Tobias Jordan
Good catch! But `BaseSegmentEncoder` is only an non-templated interface for `SegmentEncoder`, so Base should be the correct prefix.
It looks like there are many more unnecessary `hyrise::`s in the codebase :/
Yeah, but so does a case sensitive search for "hyrise::" But I'm trying to remove all unnecessary occurrences in this PR.
Now all should be gone :)
As discussed in the fornightly, a good solution is not yet found. The use of print-statements worsens the spam to the console, which in general opinion should be prohibited. It...
The current valgrind error is caused by line 219 in iterables_test.cpp. Still investigating what is wrong. The following command can be used to get to the error quick without running...
Unfortunately I just noticed the error valgrinds reports in the CI is another one than I was able to recreate on Appleton. Anyway, here you can find the error reported...
The CI's valgrind error with more information: ``` ==3445127== Command: ./clang-release-debuginfos/hyriseTest --gtest_filter=SQLiteAddIndicesTest clang-release-memcheck-test ==3445127== ==3445127== Use of uninitialised value of size 8 ==3445127== at 0x9F7E2DA: _itoa_word (_itoa.c:178) ==3445127== by 0x9F99B3D:...
Currently, the clang AddressSanitizer fails with the following error: ``` [ RUN ] PartialHashIndexTest.Iterators /usr/include/tbb/../oneapi/tbb/concurrent_hash_map.h:446:23: runtime error: downcast of misaligned address 0x000000000003 for type 'tbb::detail::d2::hash_map_iterator::node' (aka 'tbb::detail::d2::concurrent_hash_map::node'), which requires 8...
I was trying to recreate the CI error, so I used the CI docker image which uses clang 14.