Don't dump DBOptions for each column family

Summary: Currently we dump DBOptions for each column family options we dump. This leads to duplicate lines in our LOG file. This diff fixes that.

Test Plan: Check out the LOG

Reviewers: sdong, rven, yhchiang

Reviewed By: yhchiang

Subscribers: IslamAbdelRahman, yoshinorim, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D39729

db/column_family.cc

@@ -340,7 +340,7 @@ ColumnFamilyData::ColumnFamilyData(
     if (column_family_set_->NumberOfColumnFamilies() < 10) {
       Log(InfoLogLevel::INFO_LEVEL, ioptions_.info_log,
           "--------------- Options for column family [%s]:\n", name.c_str());
-      options_.Dump(ioptions_.info_log);
+      options_.DumpCFOptions(ioptions_.info_log);
     } else {
       Log(InfoLogLevel::INFO_LEVEL, ioptions_.info_log,
           "\t(skipping printing options)\n");

include/rocksdb/options.h

@@ -1049,6 +1049,8 @@ struct Options : public DBOptions, public ColumnFamilyOptions {
 
   void Dump(Logger* log) const;
 
+  void DumpCFOptions(Logger* log) const;
+
   // Set appropriate parameters for bulk loading.
   // The reason that this is a function that returns "this" instead of a
   // constructor is to enable chaining of multiple similar calls in the future.

util/options.cc

@@ -497,6 +497,10 @@ void Options::Dump(Logger* log) const {
   ColumnFamilyOptions::Dump(log);
 }   // Options::Dump
 
+void Options::DumpCFOptions(Logger* log) const {
+  ColumnFamilyOptions::Dump(log);
+}  // Options::DumpCFOptions
+
 //
 // The goal of this method is to create a configuration that
 // allows an application to write all files into L0 and