os/bluestore: disable on_applied sync_complete

1. This is causing problems in ECBackend that are not obvious.
2. It didn't show any performance win anyway.
3. I'm trying to remove on_applied callbacks entirely.  If that works out
this will all be moot!

Fixes: http://tracker.ceph.com/issues/22668
Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 70460aa99e7c2f0ddea8a8a43337ed438f4cdc5d..2de8cda682c35baa1fa3e3fa60ba2ce02027e94c 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -9205,11 +9205,7 @@ int BlueStore::queue_transactions(
     c->complete(0);
   }
   for (auto c : on_applied) {
-    // NOTE: these may complete out of order since some may be sync and some
-    // may be async.
-    if (!c->sync_complete(0)) {
-      finishers[osr->shard]->queue(c);
-    }
+    finishers[osr->shard]->queue(c);
   }
 
   logger->tinc(l_bluestore_submit_lat, ceph_clock_now() - start);