Skip to content

Commit

Permalink
fix(rewrites): change TableColumn -> Field in rewrites (#8448)
Browse files Browse the repository at this point in the history
Anyway, quick PR to remove the now renamed `ops.TableColumn`.
  • Loading branch information
gforsyth authored Feb 27, 2024
1 parent 6f2edf0 commit 3730eb6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ibis/expr/rewrites.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ def rewrite_fillna(_):

selections = []
for name in _.parent.schema.names:
col = ops.TableColumn(_.parent, name)
col = ops.Field(_.parent, name)
if (value := mapping.get(name)) is not None:
col = ops.Alias(ops.Coalesce((col, value)), name)
selections.append(col)
Expand All @@ -62,7 +62,7 @@ def rewrite_fillna(_):
def rewrite_dropna(_):
"""Rewrite DropNa expressions to use more common operations."""
if _.subset is None:
columns = [ops.TableColumn(_.parent, name) for name in _.parent.schema.names]
columns = [ops.Field(_.parent, name) for name in _.parent.schema.names]
else:
columns = _.subset

Expand Down

0 comments on commit 3730eb6

Please sign in to comment.