search

parcio / julea

A Flexible Storage Framework for HPC
GNU Lesser General Public License v3.0
33 stars 31 forks source link

changing the db-null-backend #72

Closed Qualenritter closed 4 years ago

Qualenritter commented 4 years ago

such that compareable benchmarks are possible. Especially this ensures, that the client code behaves as usual, only the backend storage is 'null'

michaelkuhn commented 4 years ago

Merged in ff80c838817c4141467fcdc1b9f4c7ad0c4b5ade and fixed up in the follow-up commits, thanks!

However, there still is a memory leak that prohibits us from enabling the new backend for tests:

TEST: .../julea/build/test/julea-test... (pid=166258)
  /db/all:                                                             OK

=================================================================
==166258==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 128 byte(s) in 1 object(s) allocated from:
    #0 0x7f59333ccc58 in __interceptor_malloc (/lib64/libasan.so.5+0x10dc58)
    #1 0x7f5932a9302e in bson_malloc (.../lib/libbson-1.0.so.0+0x1b02e)
    #2 0x7f5932f9755a in j_backend_db_func_exec ../lib/db/jdb-internal.c:99
    #3 0x7f5932f9a220 in j_db_query_exec ../lib/db/jdb-internal.c:421
    #4 0x7f5932f2c37a in j_batch_execute_same ../lib/core/jbatch.c:240
    #5 0x7f5932f2d5c6 in j_batch_execute_internal ../lib/core/jbatch.c:487
    #6 0x7f5932f2c57b in j_batch_execute ../lib/core/jbatch.c:280
    #7 0x7f5932f9e56a in j_db_iterator_new ../lib/db/jdb-iterator.c:78
    #8 0x40aed4 in iterator_get ../test/db/db.c:264
    #9 0x40c9fd in test_db_all ../test/db/db.c:387
    #10 0x7f5932d1cb41 in test_case_run .../glib/gtestutils.c:2255
    #11 0x7f5932d1cb41 in g_test_run_suite_internal .../glib/gtestutils.c:2339

SUMMARY: AddressSanitizer: 128 byte(s) leaked in 1 allocation(s).
FAIL: .../julea/build/test/julea-test

Edit: Fixed the memory leak in 29ae236f13e2cfd1081c4fb4973e8f0448618f89.