Repository navigation
fix: columns with dots or brackets in their names are no longer reported missing - #1762
Shahzeb-DataEngineer wants to merge 1 commit into
Conversation
…checks Field resolution interpreted a column name as a nested path, splitting on "." and treating "[...]" as array or map hops, so a literal column named "Rate.x" or "Amount [A]" never matched and was reported missing. Resolution now prefers a literal column of that name before reading the name as a path. Signed-off-by: Shahzeb Ali <126076773+Shahzeb-DataEngineer@users.noreply.github.com>
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
This PR seems to be part of a code generation pipeline with little human oversight and is closed preemptively, as such contributions are typically consuming much maintainer time with little gain. Note that we generally accept AI-assisted contributions, but we expect contributors to personally understand the issue, decide for a proper fix, and put reasonable effort into verifying the AI-generated code. If you want to suggest an improvement or bug fix, please create an issue. Pull Requests by your account in the next 2 weeks are likely to get closed as well. (This message was sent automatically.) |
Fixes #1760. Field resolution in the ibis check executor interpreted column names as nested paths, so a literal column named
Rate.xorAmount [A]never matched and was reported missing. Resolution now prefers a literal column of that name before reading the name as a path.