Skip to content

WIP: Test lmdb #1259

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

WIP: Test lmdb #1259

wants to merge 2 commits into from

Conversation

Smartkras
Copy link
Collaborator

WIP: test lmdb porting with OLP_SDK_ENABLE_DEFAULT_CACHE_LMDB on.

lmdb is added to external libraries. CMakeLists.txt.lmdb.in and
lmdbConfig.cmake.in was created for lmdb.
Changes in olp-cpp-sdk-core and external CMakeLists.txt was made.
Added OLP_SDK_ENABLE_DEFAULT_CACHE_LMDB option to be able to
enable/disable build of lmdb. README.md was changed with
description of a new flag.

Resolves: OLPEDGE-2645

Signed-off-by: Yevhen Krasilnyk <ext-yevhen.krasilnyk@here.com>
@Smartkras Smartkras force-pushed the test_lmdb branch 2 times, most recently from bc096ce to 21ca84b Compare October 6, 2021 07:28
@codecov
Copy link

codecov bot commented Oct 6, 2021

Codecov Report

Merging #1259 (065543a) into master (25c48da) will not change coverage.
The diff coverage is n/a.

❗ Current head 065543a differs from pull request most recent head 9119b13. Consider uploading reports for the commit 9119b13 to get more accurate results
Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1259   +/-   ##
======================================
  Coverage    85.3%   85.3%           
======================================
  Files         361     361           
  Lines       12939   12939           
======================================
  Hits        11032   11032           
  Misses       1907    1907           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25c48da...9119b13. Read the comment docs.

@Smartkras Smartkras force-pushed the test_lmdb branch 13 times, most recently from 3b1dbdd to 5515ea1 Compare October 8, 2021 07:32
lmdb is added to external libraries. CMakeLists.txt.lmdb.in and
lmdbConfig.cmake.in was created for lmdb.
Changes in olp-cpp-sdk-core and external CMakeLists.txt was made.
Added OLP_SDK_ENABLE_DEFAULT_CACHE_LMDB option to be able to
enable/disable build of lmdb. README.md was changed with
description of a new flag.

Relates-To: OLPEDGE-2645

Signed-off-by: Yevhen Krasilnyk <ext-yevhen.krasilnyk@here.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant