| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
* add "std::" prefix in headers
* add "using" declarations in .cc files.
so we don't rely on "using namespace std" in one or more included
headers.
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
for two reasons:
- better performance when looking on in the return map if the key
is not a string, as we don't need to create a temporary string
as the key
- improve the performance of crimson::AlienStore, as the latter
uses the transparent comparator. as, without this change, we'd
have to perform a deep copy to fill up the returned map with
its non-transparent-comparator version.
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|
|
|
|
|
|
|
|
| |
use a single `hint` to replace `hint_type` and `alloc_hint_flags` in
the anonymous union.
this helps to address the compile error reported by GCC-10
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|
|
|
|
|
|
| |
This is part of a series of commits to clean up using namespace at top
level in headers.
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
|
|
|
|
|
|
| |
for better readablity
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|
|
|
| |
Signed-off-by: Zengran Zhang <zhangzengran@sangfor.com.cn>
|
|
|
|
|
|
|
|
|
|
| |
For OP_CREATE in bluestore mean node didn't exist, so skip search form kvdb.
This function merged in 22f73ba. But because compatible problem,
it revert in 491c254.
This commit fix the compatible problem
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
|
|
|
|
|
|
| |
Transaction is needed in crimson, but not ObjectStore.
Signed-off-by: Samuel Just <rexludorum@gmail.com>
|
|
|
|
|
|
|
| |
This reverts commit 22f73ba62eb1fe337622ea2aefdcd829f8def851.
Fixes: http://tracker.ceph.com/issues/37774
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|
|
|
|
|
| |
For OP_CREATE in bluestore mean node didn't exist, so skip search form kvdb.
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
|
|
|
|
| |
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
| |
Signed-off-by: Kefu Chai <kchai@redhat.com>
|
|\
| |
| |
| |
| |
| |
| |
| | |
Reviewed-by: Jesse Williamson <jwilliamson@suse.de>
# Conflicts:
# src/cls/rbd/cls_rbd.cc
# src/cls/rbd/cls_rbd_types.cc
|
| |
| |
| |
| |
| |
| |
| | |
This is a portion of Part 1 of the namespace project: using ADL
properly in encode and decode so we can use namespaces easily in Ceph.
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
|
|/
|
|
| |
Signed-off-by: Igor Fedotov <ifedotov@suse.com>
|
|
|
|
|
|
|
| |
client is still sending write,startsync which is deprecated long back. Hence cleaning up OP_STARTSYNC
Fixes: http://tracker.ceph.com/issues/20604
Signed-off-by: Amit Kumar <amitkuma@redhat.com>
|
|
|
|
|
|
|
| |
This is only for FileStore, the only one that doesn't
already store it.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
|
| |
CID 1395137 (#1 of 1): Missing break in switch (MISSING_BREAK)
unterminated_case: The case for value ObjectStore::Transaction::OP_TRY_RENAME is not terminated by a 'break' statement.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
| |
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
|
|
|
|
| |
Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
|
|\
| |
| |
| |
| |
| | |
librados,osd: rados hints for future access pattern
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
|
| |
| |
| |
| |
| |
| | |
Pass these through to the ObjectStore.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|/
|
|
|
|
|
| |
These have been deprecated since hammer, and were only kept so that you
could do an offline upgrade from firefly. Bye bye!
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
| |
Signed-off-by: Samuel Just <sjust@redhat.com>
|
|
|
|
|
|
|
| |
Similar to collection_move_rename, except we ignore ENOENT and
don't allow different collections.
Signed-off-by: Samuel Just <sjust@redhat.com>
|
|
|
|
|
|
|
|
|
| |
This reverts part of commit
893e00bc0fd793090a35d275919ca0348e9f05b0.
[we leave KeyValueStore untouched]
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
| |
Replace deprecated use of collection_move() with collection_move_rename().
Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
|
|\
| |
| |
| | |
Reviewed-by: Samuel Just <sjust@redhat.com>
|
| |
| |
| |
| |
| |
| | |
We no longer need these for upgrades.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|/
|
|
|
|
|
|
| |
This informs the backend how many bits of the hash range will be covered
by the collection. Those bits are implicitly matched against the spg_t in
coll_t.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
| |
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
| |
You will not be missed!
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
diagnostic push|pop are not present before 4.6, so each use gives a
warning. Temporarily ignore these by disabling/reenabling -Wpragmas.
Note that this means the other disabled warnings are not scoped on
old gcc. I don't think this is worth fixing, since newer gcc will
respect the scope and show us the warnings outside of it.
Signed-off-by: Josh Durgin <jdurgin@redhat.com>
|
|
|
|
|
|
|
|
|
| |
When tbl is used (for compatibility), the Transaction::begin method need
to build all fields used by iterator. That includes: coll_index,
object_index, data_bl, op_bl, etc.)
Change-Id: I48ea74fec8d052f50da254a726a9c0dffead19bc
Signed-off-by: Dong Yuan <yuandong1222@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch add new Transaction::iterator interface according to new
encode/decode layout. The new iterator give the whole Op struct in a
single decode_op method.
All ObjectStore Impl (FileStore/MemStore/KeyValueStore) is also changed
to use the new interface.
Change-Id: I1900a6ec302890df2c4357b071e4966c26d7f037
Signed-off-by: Dong Yuan <yuandong1222@gmail.com>
|
|
|
|
| |
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
| |
No more users! Complicated semantics!
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
|
| |
Move the add+remove a move normally translates to directly into
that method.
Signed-off-by: Sage Weil <sage@redhat.com>
|
|
|
|
|
|
| |
expected number of objects.
Signed-off-by: Guang Yang (yguang@yahoo-inc.com)
|
|
|
|
| |
Signed-off-by: Guang Yang (yguang@yahoo-inc.com)
|
|
Signed-off-by: Sage Weil <sage@redhat.com>
|