You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
FWIW, hardcoding the numbers isn't really the proper way to do type variations. (Rather we should add the variation as an extension and refer to the extension's id.) However, given this is already used for default vs large, I guess adding view makes sense - and they can all be migrated at once to the proper way someday.
The idea here is for someone to research how this is supposed to work and then change the datafusion substrait implementation to do it
The text was updated successfully, but these errors were encountered:
alamb
changed the title
Type variation in substrait: HJardcoding the numbers isn't really the proper way to do type variations.
Type variation in substrait: Hardcoding the numbers isn't really the proper way to do type variations.
Sep 6, 2024
Originally posted by @Blizzara in #12199 (comment)
The idea here is for someone to research how this is supposed to work and then change the datafusion substrait implementation to do it
The text was updated successfully, but these errors were encountered: