Skip to content

fix: Wrap LocalOutputNode return value in ArrowTableValue for consist…#6286

Merged
ntkathole merged 2 commits intofeast-dev:masterfrom
cutoutsy:fix/local-output-node-return-type
Apr 16, 2026
Merged

fix: Wrap LocalOutputNode return value in ArrowTableValue for consist…#6286
ntkathole merged 2 commits intofeast-dev:masterfrom
cutoutsy:fix/local-output-node-return-type

Conversation

@cutoutsy
Copy link
Copy Markdown
Contributor

@cutoutsy cutoutsy commented Apr 16, 2026

Wrap LocalOutputNode return value in ArrowTableValue for consistent DAG


Open with Devin

@cutoutsy cutoutsy requested a review from a team as a code owner April 16, 2026 02:31
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

…ent DAG output

Signed-off-by: cutoutsy <cutoutsy@gmail.com>
…um_rows

Signed-off-by: cutoutsy <cutoutsy@gmail.com>
@ntkathole ntkathole force-pushed the fix/local-output-node-return-type branch from 7c99bf5 to fc4729a Compare April 16, 2026 14:23
@ntkathole ntkathole merged commit a16cd55 into feast-dev:master Apr 16, 2026
16 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants