Dangling ANDs after removing Condition

[antoni4040]

Hello. I'm doing a transform like this:

def removeExplicitJoins(node):
            if isinstance(node, exp.Condition):
                left = node.this
                right = node.expression
                if isinstance(left, exp.Column) and isinstance(right, exp.Column):
                    if left.table in self.untouchedTables or right.table in self.untouchedTables:
                        pass
                    else:
                        return None
            return node

But later when converting to sql text, I get syntax error because the text contains "AND AND".
If I set it to pretty, the problem dissapears, but I fear there's a bug here.

[tobymao]

it’s not safe to remove a condition. you’d have to replace it with a new one. a safer way is to replace it with exp.true() instead so that there are no hanging ands.

[antoni4040]

Then I get a bunch of AND TRUE AND TRUE AND TRUE, not very nice...

[tobymao]

you can run the optimizer.simplify command after.

or as i said, replace the entire node and not just the sub expression.