Revert set log level to DBA WARN

Co-authored-by: gaopy3 <gao.panyu@qq.com>

Author: 2 people

build.sh

@@ -157,7 +157,7 @@ function build_package
 {
     STATIC_LINK_LGPL_DEPS_OPTION=ON
     ENABLE_BOLT_OPTION=OFF
-    do_build "$@" -DOB_BUILD_PACKAGE=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo -DOB_USE_LLD=$LLD_OPTION -DENABLE_FATAL_ERROR_HANG=OFF -DENABLE_AUTO_FDO=ON -DENABLE_THIN_LTO=ON -DENABLE_HOTFUNC=ON -DENABLE_BOLT=$ENABLE_BOLT_OPTION -DOB_STATIC_LINK_LGPL_DEPS=$STATIC_LINK_LGPL_DEPS_OPTION -DDEFAULT_LOG_LEVEL=OB_LOG_LEVEL_DBA_WARN
+    do_build "$@" -DOB_BUILD_PACKAGE=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo -DOB_USE_LLD=$LLD_OPTION -DENABLE_FATAL_ERROR_HANG=OFF -DENABLE_AUTO_FDO=ON -DENABLE_THIN_LTO=ON -DENABLE_HOTFUNC=ON -DENABLE_BOLT=$ENABLE_BOLT_OPTION -DOB_STATIC_LINK_LGPL_DEPS=$STATIC_LINK_LGPL_DEPS_OPTION
 }
 
 # build - configurate project and prepare to compile, by calling make

cmake/Env.cmake

@@ -29,7 +29,6 @@ ob_define(OB_CC "")
 ob_define(OB_CXX "")
 ob_define(OB_BUILD_STANDALONE OFF)
 ob_define(OB_BUILD_LITE ON)
-ob_define(DEFAULT_LOG_LEVEL OB_LOG_LEVEL_WARN)
 
 # 'ENABLE_PERF_MODE' use for offline system insight performance test
 # PERF_MODE macro controls many special code path in system
@@ -167,8 +166,6 @@ if(BUILD_EMBED_MODE)
   add_definitions(-DOB_BUILD_EMBED_MODE)
 endif()
 
-add_definitions(-DDEFAULT_LOG_LEVEL=${DEFAULT_LOG_LEVEL})
-
 set(OB_OBJCOPY_BIN "${DEVTOOLS_DIR}/bin/objcopy")
 
 # NO RELERO: -Wl,-znorelro

deps/oblib/src/lib/oblog/ob_log.cpp

@@ -469,6 +469,7 @@ void ObLogger::print_trace_buffer(const char* mod_name,
 
 
 
+const char *const ObLogger::errstr_[] = {"ERROR", "WARN", "INFO", "EDIAG", "WDIAG", "TRACE", "DEBUG"};
 
 ObLogger::ObLogger()
   : ObBaseLogWriter(), log_file_(), max_file_size_(DEFAULT_MAX_FILE_SIZE), max_file_index_(0),

deps/oblib/src/lib/oblog/ob_log.h

@@ -631,7 +631,6 @@ class ObLogger : public ObBaseLogWriter
   int32_t get_log_level(const uint64_t par_mod_id) const;
   int32_t get_log_level(const uint64_t par_mod_id, const uint64_t sub_mod_id) const;
   inline const char *get_level_str() const { return errstr_[id_level_map_.get_level()]; }
-  static constexpr const char *get_level_str(const int8_t level) { return errstr_[level]; }
 
   void disable_thread_log_level() { disable_thread_log_level_ = true; }
 
@@ -791,7 +790,7 @@ class ObLogger : public ObBaseLogWriter
   void drop_log_items(ObIBaseLogItem **items, const int64_t item_cnt) override;
   void unlink_if_need(const char *file);
 private:
-  static constexpr const char *const errstr_[] = {"ERROR", "WARN", "INFO", "EDIAG", "WDIAG", "TRACE", "DEBUG"};
+  static const char *const errstr_[];
   // default log rate limiter if there's no tl_log_limiger
   static ::oceanbase::lib::ObRateLimiter *default_log_limiter_;
   RLOCAL_STATIC(lib::ObRateLimiter*, tl_log_limiter_);

src/observer/embed/python/ob_embed_impl.cpp

@@ -220,7 +220,7 @@ int ObLiteEmbed::do_open_(const char* db_dir, int64_t port)
     MPRINT("db %s opened by other process", db_dir);
   } else if (FALSE_IT(pid_locked = true)) {
   } else {
-    OB_LOGGER.set_log_level(DEFAULT_LOG_LEVEL);
+    OB_LOGGER.set_log_level("INFO");
     ObSqlString log_file;
     if (OB_FAIL(log_file.assign_fmt("%s/log/seekdb.log", opts.base_dir_.ptr()))) {
       MPRINT("calculate log file failed %d", ret);

src/observer/main.cpp

@@ -310,7 +310,7 @@ int inner_main(int argc, char *argv[])
   setlocale(LC_TIME, "en_US.UTF-8");
   setlocale(LC_NUMERIC, "en_US.UTF-8");
 
-  opts->log_level_ = DEFAULT_LOG_LEVEL;
+  opts->log_level_ = OB_LOG_LEVEL_WARN;
   if (FAILEDx(parse_args(argc, argv, *opts))) {
   }
 

src/observer/ob_command_line_parser.cpp

@@ -203,9 +203,9 @@ int ObCommandLineParser::handle_option(int option, const char* value, ObServerOp
         MPRINT("Invalid argument, the value should not be empty");
       } else {
         if (OB_FAIL(OB_LOGGER.level_str2int(value, opts.log_level_))) {
-          MPRINT("Invalid log level. Back to default log level.");
+          MPRINT("Invalid log level. Back to INFO log level.");
           ret = OB_SUCCESS;
-          opts.log_level_ = DEFAULT_LOG_LEVEL;
+          opts.log_level_ = OB_LOG_LEVEL_WARN;
         }
       }
       break;

src/share/parameter/ob_parameter_seed.ipp

@@ -202,7 +202,7 @@ DEF_INT(cluster_id, OB_CLUSTER_PARAMETER, "1", "[1,4294901759]", "ID of the clus
         ObParameterAttr(Section::OBSERVER, Source::DEFAULT, EditLevel::DYNAMIC_EFFECTIVE));
 DEF_STR(obconfig_url, OB_CLUSTER_PARAMETER, "", "URL for OBConfig service",
         ObParameterAttr(Section::OBSERVER, Source::DEFAULT, EditLevel::DYNAMIC_EFFECTIVE));
-DEF_LOG_LEVEL(syslog_level, OB_CLUSTER_PARAMETER, ObLogger::get_level_str(DEFAULT_LOG_LEVEL), "specifies the current level of logging. There are DEBUG, TRACE, WDIAG, EDIAG, INFO, WARN, ERROR, seven different log levels.",
+DEF_LOG_LEVEL(syslog_level, OB_CLUSTER_PARAMETER, "WDIAG", "specifies the current level of logging. There are DEBUG, TRACE, WDIAG, EDIAG, INFO, WARN, ERROR, seven different log levels.",
               ObParameterAttr(Section::OBSERVER, Source::DEFAULT, EditLevel::DYNAMIC_EFFECTIVE),
              "DEBUG, TRACE, WDIAG, EDIAG, INFO, WARN, ERROR");
 DEF_STR_WITH_CHECKER(alert_log_level, OB_CLUSTER_PARAMETER, "INFO",