CLN: Remove debugging code

Author: zbrookle

dataframe_sql/parsing/sql_parser.py

@@ -1229,7 +1229,6 @@ def handle_non_token_non_tree(query_info: QueryInfo, token, token_pos):
             # TODO Get rid of collecting this alias information since its part of the
             #  column object
             if token.alias:
-                print(query_info.aliases)
                 query_info.aliases[token.name] = token.alias
 
         if isinstance(token, Expression):
@@ -1409,7 +1408,6 @@ def handle_columns(
         :param internal_transformer: Transformer to transform the where clauses
         :return:
         """
-        print(columns)
         where_value = None
         where_plan = ":"
         if where_expr is not None:
@@ -1444,7 +1442,6 @@ def handle_columns(
                 else:
                     final_names.append(column.name)
 
-            print(final_names)
             if where_value is not None:
                 new_frame = first_frame.loc[where_value, column_names]
             else:

dataframe_sql/sql_objects.py

@@ -246,21 +246,27 @@ def __repr__(self):
 
     def __or__(self, other):
         if not isinstance(other, Value):
-            raise Exception(f"Operator | is not supported between type {type(other)} "
-                            f"and type ValueWithPlan")
-
-        return ValueWithPlan(self.get_value() | other.get_value(),
-                             f"{self.get_plan_representation()} | "
-                             f"{other.get_plan_representation()}")
+            raise Exception(
+                f"Operator | is not supported between type {type(other)} "
+                f"and type ValueWithPlan"
+            )
+
+        return ValueWithPlan(
+            self.get_value() | other.get_value(),
+            f"{self.get_plan_representation()} | " f"{other.get_plan_representation()}",
+        )
 
     def __and__(self, other):
         if not isinstance(other, Value):
-            raise Exception(f"Operator && is not supported between type {type(other)} "
-                            f"and type ValueWithPlan")
-
-        ValueWithPlan(self.get_value() & other.get_value(),
-                      f"{self.get_plan_representation()} & "
-                      f"{other.get_plan_representation()}")
+            raise Exception(
+                f"Operator && is not supported between type {type(other)} "
+                f"and type ValueWithPlan"
+            )
+
+        ValueWithPlan(
+            self.get_value() & other.get_value(),
+            f"{self.get_plan_representation()} & " f"{other.get_plan_representation()}",
+        )
 
     def get_plan_representation(self) -> str:
         return self.execution_plan

dataframe_sql/sql_select_query.py

@@ -123,7 +123,6 @@ def __init__(self, sql: str, show_exececution_plan: bool = False):
     def parse_sql(self):
         try:
             tree = self.parser.parse(self.sql)
-            print(tree.pretty())
 
             table_info = TableInfo()
 

dataframe_sql/tests/pandas_sql_functionality_test.py

@@ -1429,6 +1429,7 @@ def test_math_order_of_operations_with_parens():
 
     tm.assert_frame_equal(pandas_frame, my_frame)
 
+
 def test_boolean_order_of_operations_with_parens():
     """
     Test boolean order of operations with parentheses
@@ -1440,13 +1441,18 @@ def test_boolean_order_of_operations_with_parens():
         "(month = 'nov' and day = 'tue')"
     )
 
+    pandas_frame = FOREST_FIRES.copy()
+    pandas_frame = pandas_frame[((pandas_frame['month'] == 'oct') &
+                                 (pandas_frame['day'] == 'fri')) |
+                                ((pandas_frame['month'] == 'nov') &
+                                 (pandas_frame['day'] == 'tue'))].reset_index(drop=True)
 
-
+    tm.assert_frame_equal(pandas_frame, my_frame)
 
 
 if __name__ == "__main__":
     register_env_tables()
 
-    test_right_joins()
+    test_boolean_order_of_operations_with_parens()
 
     remove_env_tables()