feat(core)!: bump to substrait v0.87.0 (toleration)#797
feat(core)!: bump to substrait v0.87.0 (toleration)#797mbwhite wants to merge 1 commit intosubstrait-io:mainfrom
Conversation
Signed-off-by: MBWhite <whitemat@uk.ibm.com>
|
Are you sure this bump doesn't bring in spec changes that need to be accounted for? |
|
Good question @benbellick so I went back and double checked. As far as I can see no changes are needed directly to accommodate the changes; corrections to the extension definitions will be picked up when they are used. (I've tested this locally - which to be transparent is my motivation!) It is possible that changes could be made that would have not been possible before; though those probably best in their own PRs |
|
I checked the diff and there are a few substrait spec changes that we should handle:
No need to address all of these things here, but I think for good hygiene of this repo (and all of the substrait repos) it would be good to open tickets for these things if they don't exist already so that we can identify that they are spec gaps. Thanks! |
|
I had created issues across the 4 active libraries for this spec upgrade such as this one: #790 I tried to list out the modifications introduced in this spec version. Feel free to amend if you see something missing. |
|
thanks @benbellick @nielspardon I've updated the title to indicate that this is toleration only #790 will be need for full support. Though please do close this if you'd rather it done 'all-in-one' |
Bump up to the latest version of the Substrait spec.
No code changes.