New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fast denc encoding #11027
fast denc encoding #11027
Conversation
return iterator(this, false); | ||
} | ||
iterator begin_deep() { | ||
return iterator(this, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/false/true/
f727ab9
to
0fe3cef
Compare
// itype == internal type | ||
// otype == external type, i.e., the type on the wire | ||
|
||
// NOTE: set supported == 2 instead of true. This prevents these from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be useful to have a place in the comments where the values of the supported, featured, bounded, etc. are explicitly documented in a table.
Also the definition of "featured object" isn't immediately clear (and quite misleading). One line of comment where the term is first introduced would be hugely helpful.
@liewegas With first 4 commits (sharedblob::put() one causing crash) , memory leaks are resolved in my setup. |
37f9cd8
to
d9dab11
Compare
This looks about right to me. |
14b176b
to
b9c4380
Compare
…n terms of varint Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
#include it from encoding.h so that we can make old STL container helpers conditional on a new-style helper not being available. Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
The bufferlist one is for internal use only. Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
The BlueRocksEnv uses the append(const char *, size_t) appender. Ensure that this data is copied into a page-aligned buffer, and that we "pad" the write out with the remainder of the page. Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Do the copy up-front. This is useful if we know our target buffer must be a contiguous buffer (e.g., because it will be passed to a kv store). Signed-off-by: Sage Weil <sage@redhat.com>
Avoid an inevitable rebuild() since we have to pass this to rocksdb as a single memory buffer anyway. Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Back in 46522cf we broke the newline placement for the * lines. Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
No description provided.