First ordinals refactor PR ready for review! Decided to take on modifying the schema struct first. @Cory Grinstead @R. C. Howell https://github.com/Eventual-Inc/Daft/pull/4204

Heroic effort, the internalization looks good and we have some opportunities to improve the abstractions/helpers. No blocking comments, only thing to consider at the moment is an error variant for cause "ambiguous" rather than using "not found"

Ah yeah that would make sense. And yeah, still got a few bugs to squash haha

Heroic effort

😄 😄 😄

Oh btw @R. C. Howell @Cory Grinstead is this good to merge?

Yes all good on my end 🚢

done