fix analyze_parameter/analyze_variable log (#338)

* table dump print pretty result * Fix regression testing bugs * Fix regression testing bugs * Optimize logs * Optimize logs * Optimize logs * fix: gather tabledump * fix: gather tabledump * fix analyze flt_trace offline * remove duplicate code & optimize log * remove duplicate code & optimize log * fix: command auto-completion * fix: command auto-completion * fix: command auto-completion * fix analyze_parameter/analyze_variable log
oceanbase · Jul 16, 2024 · e5a9f00 · e5a9f00
1 parent 21e94e2
commit e5a9f00
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/handler/analyzer/analyze_parameter.py b/handler/analyzer/analyze_parameter.py
@@ -135,7 +135,7 @@ def init_option_diff(self):
         if offline_file_option:
             if not os.path.exists(os.path.abspath(offline_file_option)):
                 self.stdio.error('args --file [{0}] not exist: No such file, Please specify it again'.format(os.path.abspath(offline_file_option)))
-                exit(-1)
+                return False
             else:
                 self.parameter_file_name = os.path.abspath(offline_file_option)
                 self.check_file_valid()

diff --git a/handler/analyzer/analyze_variable.py b/handler/analyzer/analyze_variable.py
@@ -83,13 +83,13 @@ def init_option(self):
         if offline_file_option:
             if not os.path.exists(os.path.abspath(offline_file_option)):
                 self.stdio.error('args --file [{0}] not exist: No such file, Please specify it again'.format(os.path.abspath(offline_file_option)))
-                exit(-1)
+                return False
             else:
                 self.variable_file_name = os.path.abspath(offline_file_option)
                 self.check_file_valid()
         else:
             self.stdio.error("args --file need provided to find the parts where variables have changed.")
-            exit(-1)
+            return False
 
         if store_dir_option and store_dir_option != "./":
             if not os.path.exists(os.path.abspath(store_dir_option)):