Keyvi server #117
Merged
Keyvi server #117
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added docs to obtain simple statistics from a compiled keyvifile
disable MAP_POPULATE on Mac
…c_os fixed MAP_POPULATE usage on MacOS
…y impl methods _value_iterator_wrapper() and _item_iterator_wrapper()) to return decoded values() - added unit tests for Dictionary methods GetAllKeys(), GetAllValues() and GetAllItems() - updated auto-generated files pykeyvi.pyx and pykeyvi.cpp
…alue. - Fixed cython Match::GetValue method to use GetMsgPackedValueAsString, which will be faster and without dependencies on zlib and snappy - removed cython code dependencies on zlib and snappy
- made it lightwight by holding just shard_ptr to EntryIterator instead of having member, so there is not need to copy/move hevy EntryIterator object during heap balancing. (also this will from prevent heap defragmentation) benchmarks shown about overall about 8 times merging speed improvement
Fixed error in README.
Added a note about TMPDIR
…s_fix - fixed Dictionary methods GetAllValues() and GetAllItems()
# Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
Dictionary Merger optimization: now it merges dictionaries about 8 times faster
…ge_elimination_from_cython eliminated pykeyvi dependencies on snappy and zlib
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
updates from master