fix(rewrites): change TableColumn -> Field in rewrites

[gforsyth]

---

Description of changes

I don't know if just aren't using these rewrites or aren't testing them.

Anyway, quick PR to remove the now renamed ops.TableColumn -- if it makes more sense to rip out the rewrites, I can do that instead.

Issues closed

[github-actions]

ACTION NEEDED

Ibis follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message.

Please update your PR title and description to match the specification.

[kszucs]

Apparently we don't use rewrite_dropna and rewrite_fillna anymore. We should check whether there are backends not supporting FillNa and DropNa and if there are, we can keep these rewrites.

[jcrist]

Yeah, these got dropped in the refactor. I think we might want to bring them back though instead of implementing in the compiler (see

ibis/ibis/backends/sql/compiler.py

Lines 1319 to 1336 in e4edc78

	def visit_FillNa(self, op, *, parent, replacements):
	if isinstance(replacements, Mapping):
	mapping = replacements
	else:
	mapping = {
	name: replacements
	for name, dtype in op.schema.items()
	if dtype.nullable
	}
	exprs = {
	col: (
	self.f.coalesce(sg.column(col, quoted=self.quoted), sge.convert(alt))
	if (alt := mapping.get(col)) is not None
	else sg.column(col, quoted=self.quoted)
	)
	for col in op.schema.keys()
	}
	return sg.select(*self._cleanup_names(exprs)).from_(parent)

) since doing this via a rewrite may allow fusing selects to generate nicer SQL. Doing this during compilation doesn't allow for that.

That said, happy to merge the fix as is (or delete the rewrites fully until/if we want them back).

[gforsyth]

I can look at restoring the expr-level rewrites

[kszucs]

I think we might want to bring them back though instead of implementing in the compiler

SGTM, could you create a ticket to track this? I think we can merge it as is.

[gforsyth]

SGTM, could you create a ticket to track this? I think we can merge it as is.

will do