Remove unnecessary trailing semicolons from Python scripts. (#15976)

Ref: https://pylint.pycqa.org/en/latest/user_guide/messages/warning/unnecessary-semicolon.html
abner-ma · Sep 6, 2023 · 5e5295a · 5e5295a
1 parent 4e865bb
commit 5e5295a
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 10 deletions.
diff --git a/gpMgmt/bin/gpload.py b/gpMgmt/bin/gpload.py
@@ -807,11 +807,11 @@ def run(self):
 
 def cli_help():
     help_path = os.path.join(sys.path[0], '..', 'docs', 'cli_help', EXECNAME +
-                             '_help');
+                             '_help')
     f = None
     try:
         try:
-            f = open(help_path);
+            f = open(help_path)
             return f.read(-1)
         except:
             return ''
@@ -1278,7 +1278,7 @@ def log(self, level, a):
                 pass
 
         if level == self.ERROR:
-            self.exitValue = 2;
+            self.exitValue = 2
             sys.exit(self.exitValue)
 
     def getconfig(self, a, typ=None, default='error', extraStuff='', returnOriginal=False):
@@ -2593,7 +2593,7 @@ def count_errors(self):
                     results = cur.fetchall()
                     global NUM_WARN_ROWS
                     NUM_WARN_ROWS = (results[0])[0]
-                    return (results[0])[0];
+                    return (results[0])[0]
         return 0
 
     def report_errors(self):

diff --git a/gpMgmt/bin/gppylib/commands/gp.py b/gpMgmt/bin/gppylib/commands/gp.py
@@ -341,7 +341,7 @@ def __init__(self, name, dataDir, port, era,
 
         # build pg_ctl command
         c = PgCtlStartArgs(dataDir, b, era, wrapper, wrapper_args, wait, timeout)
-        logger.info("CoordinatorStart pg_ctl cmd is %s", c);
+        logger.info("CoordinatorStart pg_ctl cmd is %s", c)
         self.cmdStr = str(c)
 
         Command.__init__(self, name, self.cmdStr, ctxt, remoteHost)
@@ -398,7 +398,7 @@ def __init__(self, name, gpdb, numContentsInCluster, era, mirrormode,
 
         # build pg_ctl command
         c = PgCtlStartArgs(datadir, b, era, wrapper, wrapper_args, pg_ctl_wait, timeout)
-        logger.info("SegmentStart pg_ctl cmd is %s", c);
+        logger.info("SegmentStart pg_ctl cmd is %s", c)
         self.cmdStr = str(c) + ' 2>&1'
 
         Command.__init__(self, name, self.cmdStr, ctxt, remoteHost)

diff --git a/gpMgmt/bin/gppylib/commands/unix.py b/gpMgmt/bin/gppylib/commands/unix.py
@@ -737,6 +737,6 @@ def getDescendentProcesses(pid):
 elif curr_platform == FREEBSD:
     SYSTEM = FreeBsdPlatform()
 elif curr_platform == OPENBSD:
-    SYSTEM = OpenBSDPlatform();
+    SYSTEM = OpenBSDPlatform()
 else:
     raise Exception("Platform %s is not supported.  Supported platforms are: %s", SYSTEM, str(platform_list))
diff --git a/gpMgmt/bin/gppylib/gpparseopts.py b/gpMgmt/bin/gppylib/gpparseopts.py
@@ -60,7 +60,7 @@ def read_helpfile(self):
         f = None
         try:
             try:
-                f = open(help_path);
+                f = open(help_path)
                 return f.read(-1)
             except:
                 return None

diff --git a/gpMgmt/bin/gppylib/programs/clsSystemState.py b/gpMgmt/bin/gppylib/programs/clsSystemState.py
@@ -187,7 +187,7 @@ def addValue(self, key, value, isWarning=False):
         self.__currentSegmentData["values"][key] = value
         self.__currentSegmentData["isWarning"][key] = isWarning
 
-        assert key in self.__allValues;
+        assert key in self.__allValues
 
     def isClusterProbablyDown(self, gpArray):
         """

diff --git a/gpcontrib/gp_replica_check/gp_replica_check.py b/gpcontrib/gp_replica_check/gp_replica_check.py
@@ -70,7 +70,7 @@ def __init__(self, segrow, datname, relation_types):
         self.ploc = segrow[4]
         self.mloc = segrow[5]
         self.datname = datname
-        self.relation_types = relation_types;
+        self.relation_types = relation_types
         self.result = False
         self.lock = threading.Lock()