summaryrefslogtreecommitdiffstats
path: root/src/rgw
diff options
context:
space:
mode:
authorYehuda Sadeh <yehuda@redhat.com>2017-11-07 00:06:20 +0100
committerYehuda Sadeh <yehuda@redhat.com>2018-04-10 17:05:39 +0200
commitb2d16317ba30b4d59662b79d01ac8f58697b7608 (patch)
tree5a57d727b49054443f2f893126731b3a1d989013 /src/rgw
parentrgw: don't allow for io_id to complete more than once (diff)
downloadceph-b2d16317ba30b4d59662b79d01ac8f58697b7608.tar.xz
ceph-b2d16317ba30b4d59662b79d01ac8f58697b7608.zip
rgw: remove warning
No need to clean io_ids when stack finishes, as list of io_ids is kept inside the stack object, and will be cleaned up anyway. We also have a list of finished ios in the completion manager (complete_reqs), but this is a fifo, and will be cleaned up once pulled out (and we identify that io references an obsolete stack). Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Diffstat (limited to 'src/rgw')
-rw-r--r--src/rgw/rgw_coroutine.cc1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/rgw/rgw_coroutine.cc b/src/rgw/rgw_coroutine.cc
index 4e3fed07d8d..c93ff816d19 100644
--- a/src/rgw/rgw_coroutine.cc
+++ b/src/rgw/rgw_coroutine.cc
@@ -488,7 +488,6 @@ void RGWCoroutinesStack::init_new_io(RGWIOProvider *io_provider)
bool RGWCoroutinesStack::try_io_unblock(const rgw_io_id& io_id)
{
if (!can_io_unblock(io_id)) {
-#warning io_finish_ids needs to be cleaned up when owning stack finishes
auto p = io_finish_ids.emplace(io_id.id, io_id);
auto& iter = p.first;
bool inserted = p.second;