]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: re-indent some code in _process_pg_info
authorColin Patrick McCabe <cmccabe@alumni.cmu.edu>
Tue, 23 Nov 2010 01:31:50 +0000 (17:31 -0800)
committerColin Patrick McCabe <cmccabe@alumni.cmu.edu>
Tue, 23 Nov 2010 01:35:12 +0000 (17:35 -0800)
Re-indent the code and add a comment.

Signed-off-by: Colin McCabe <colinm@hq.newdream.net>
src/osd/OSD.cc

index 4e442555715f7298d9ff1f285e3666452d17293e..05704f5e608cb516601e951bb30c3f2f9902520f 100644 (file)
@@ -3930,41 +3930,41 @@ void OSD::_process_pg_info(epoch_t epoch, int from,
       pg->update_stats();
       do_queries(query_map);
     }
-  } else {
-    if (!pg->info.dne()) {
-      // i am REPLICA
-      if (!pg->is_active()) {
-       pg->merge_log(*t, info, log, from);
-       pg->activate(*t, fin->contexts, info_map);
+  } else if (!pg->info.dne()) {
+    if (!pg->is_active()) {
+      // INACTIVE REPLICA
+      pg->merge_log(*t, info, log, from);
+      pg->activate(*t, fin->contexts, info_map);
+    } else {
+      // ACTIVE REPLICA
+
+      // just update our stats
+      dout(10) << *pg << " writing updated stats" << dendl;
+      pg->info.stats = info.stats;
+
+      // did a snap just get purged?
+      if (info.purged_snaps.size() < pg->info.purged_snaps.size()) {
+       stringstream ss;
+       ss << "pg " << pg->info.pgid << " replica got purged_snaps " << info.purged_snaps
+          << " had " << pg->info.purged_snaps;
+       logclient.log(LOG_WARN, ss);
+       pg->info.purged_snaps = info.purged_snaps;
       } else {
-       // just update our stats
-       dout(10) << *pg << " writing updated stats" << dendl;
-       pg->info.stats = info.stats;
-
-       // did a snap just get purged?
-       if (info.purged_snaps.size() < pg->info.purged_snaps.size()) {
-         stringstream ss;
-         ss << "pg " << pg->info.pgid << " replica got purged_snaps " << info.purged_snaps
-            << " had " << pg->info.purged_snaps;
-         logclient.log(LOG_WARN, ss);
+       interval_set<snapid_t> p = info.purged_snaps;
+       p.subtract(pg->info.purged_snaps);
+       if (!p.empty()) {
+         dout(10) << " purged_snaps " << pg->info.purged_snaps
+                  << " -> " << info.purged_snaps
+                  << " removed " << p << dendl;
+         snapid_t sn = p.range_start();
+         coll_t c(info.pgid, sn);
+         t->remove_collection(c);
+
          pg->info.purged_snaps = info.purged_snaps;
-       } else {
-         interval_set<snapid_t> p = info.purged_snaps;
-         p.subtract(pg->info.purged_snaps);
-         if (!p.empty()) {
-           dout(10) << " purged_snaps " << pg->info.purged_snaps
-                    << " -> " << info.purged_snaps
-                    << " removed " << p << dendl;
-           snapid_t sn = p.range_start();
-           coll_t c(info.pgid, sn);
-           t->remove_collection(c);
-           
-           pg->info.purged_snaps = info.purged_snaps;
-         }
        }
-
-       pg->write_info(*t);
       }
+
+      pg->write_info(*t);
     }
   }