pylint fixes

Signed-off-by: Dinesh Dutt <[email protected]>

Author: ddutt

suzieq/db/parquet/parquetdb.py

@@ -130,8 +130,7 @@ def read(self, table_name: str, data_format: str,
                         continue
                     if need_sqvers:
                         vers = float(str(elem).split('=')[-1])
-                        if vers > max_vers:
-                            max_vers = vers
+                        max_vers = max(vers, max_vers)
 
                     dataset = ds.dataset(elem, format='parquet',
                                          partitioning='hive')
@@ -539,8 +538,7 @@ def _get_cp_dataset(self, table_name: str, need_sqvers: bool,
                 continue
             if need_sqvers:
                 vers = float(str(elem).split('=')[-1])
-                if vers > max_vers:
-                    max_vers = vers
+                max_vers = max(vers, max_vers)
 
             dataset = ds.dataset(elem, format='parquet', partitioning='hive')
 
@@ -631,7 +629,6 @@ def check_ns_conds(ns_to_test: str, filter_list: List[str],
             Returns:
                 bool: the result of the expression
             """
-            # pylint: disable=comparison-with-callable
             # We would like to init the result to False if we concat the
             # expressions with OR, while with True if we use AND.
             res = False

suzieq/engines/pandas/routes.py

@@ -35,7 +35,6 @@ def _cons_addnl_fields(self, columns: List[str], addnl_fields: List[str],
 
         return addnl_fields
 
-    # pylint: disable=too-many-statements
     def get(self, **kwargs):
         '''Return the routes table for the given filters'''
 

suzieq/gui/stlit/xplore.py

@@ -389,8 +389,7 @@ def _draw_aggrid_df(self, df) -> AgGrid:
 
         start_row = self._state.start_row
         end_row = self._state.end_row
-        if end_row > df.shape[0]:
-            end_row = df.shape[0]
+        end_row = min(end_row, df.shape[0])
 
         if end_row - start_row > MAX_ROW_WARN:
             st.warning('Printing too many rows can fail or be very slow')