chore: update to substrait 0.81.0#675
Conversation
|
I'm in support of this, but let's wait until the upstream is resolved before merging this IMO. EDIT: Also, when the upstream settles, we will have to bump the version of substait here, and then this file will become a compilation error :) |
Fine with me. Just found this file in my research to understand whether this is used anywhere and thought I prepare the PR before I forget about it. |
|
substrait-io/substrait#940 has been merged |
Signed-off-by: Niels Pardon <par@zurich.ibm.com>
1db29e2 to
0e10a81
Compare
|
Nice, thanks for doing this! I don't know if this is the best place to raise this discussion, but I'm wondering if we should try and make a point moving forwards of opening tickets for all of the relevant upstream substrait changes so that we don't forget to tackle them. This is not relevant for this PR because
Just making a note for the future! Thanks 🚀 |
I created a thread in the Substrait Slack in #substrait-java. |
related to substrait-io/substrait#940 and substrait-io/substrait#849
core/src/main/java/io/substrait/type/proto/ParameterizedProtoConverter.javawhich is no longer part of the speccore/src/main/java/io/substrait/type/proto/TypeExpressionProtoVisitor.javawhich is no longer part of the spec