From: David Zafman <dzafman@redhat.com>
Date: Fri, 6 Mar 2015 03:16:45 +0000 (-0800)
Subject: osd: FileJournal: Add _fdump() that takes Formatter instead of ostream
X-Git-Tag: v9.0.2~168^2~27
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=130eba48495d0e6acd8a472cbf0c47ebbc7a038c;p=ceph.git

osd: FileJournal: Add _fdump() that takes Formatter instead of ostream

Signed-off-by: David Zafman <dzafman@redhat.com>
---

diff --git a/src/os/FileJournal.cc b/src/os/FileJournal.cc
index 136de61d0ff0..4ba468f7e13f 100644
--- a/src/os/FileJournal.cc
+++ b/src/os/FileJournal.cc
@@ -583,7 +583,15 @@ int FileJournal::simple_dump(ostream& out)
 
 int FileJournal::_dump(ostream& out, bool simple)
 {
-  dout(10) << "dump" << dendl;
+  JSONFormatter f(true);
+  int ret = _fdump(f, simple);
+  f.flush(out);
+  return ret;
+}
+
+int FileJournal::_fdump(Formatter &f, bool simple)
+{
+  dout(10) << "_fdump" << dendl;
 
   assert(fd == -1);
   int err = _open(false, false);
@@ -598,7 +606,6 @@ int FileJournal::_dump(ostream& out, bool simple)
 
   off64_t next_pos = header.start;
 
-  JSONFormatter f(true);
   f.open_object_section("journal");
 
   f.open_object_section("header");
@@ -669,7 +676,6 @@ int FileJournal::_dump(ostream& out, bool simple)
 
   f.close_section();
   f.close_section();
-  f.flush(out);
   dout(10) << "dump finish" << dendl;
 
   close();
diff --git a/src/os/FileJournal.h b/src/os/FileJournal.h
index d565252f4efd..9e07b4046d97 100644
--- a/src/os/FileJournal.h
+++ b/src/os/FileJournal.h
@@ -415,6 +415,7 @@ private:
 
   int dump(ostream& out);
   int simple_dump(ostream& out);
+  int _fdump(Formatter &f, bool simple);
 
   void flush();