Latest substrait patch release does not compile against DF

[westonpace]

Describe the bug

Substrait released 0.62.3 which unfortunately has breaking changes from 0.62.2. Datafusion depends on 0.62 and so it will allow 0.62.3 but will fail to compile.

To Reproduce

Check out a clean repo and try to compile datafusion-substrait.

Expected behavior

Compiles correctly

Additional context

The error is:

error[E0004]: non-exhaustive patterns: `&RexType::Lambda(_)` and `&RexType::LambdaInvocation(_)` not covered

[KARTIK64-rgb]

Hi @westonpace!

I'd like to work on this issue. The fix seems straightforward adding exhaustive match arms for &RexType::Lambda(_) and &RexType::LambdaInvocation(_) in the relevant match expressions within datafusion-substrait.

Could I be assigned to this issue? Thanks!

[westonpace]

Go for it. I think there is one other compile error I didn't list in the original description (the field type_reference was removed from UserDefined) so it might be slightly more but hopefully shouldn't be too bad.

Could I be assigned to this issue?

From the contributor's guide

If you want to work on an issue which is not already assigned to someone else and there are no comment indicating that someone is already working on that issue then you can assign the issue to yourself by submitting a single word comment take. This will assign the issue to yourself. However, if you are unable to make progress you should unassign the issue by commenting a single word untake.

[KARTIK64-rgb]

take

[benbellick]

Copying my comment from here:

This release of substrait-rs breaks semvar compatibility. This was an oversight on my end due to the fact that release-plz didn't detect a submodule bump as a breaking change.
We are investigating performing cargo yank on this release. Would that solution work for others here?

[milenkovicm]

i believe, we can close this as problematic version has been yanked
#20785 (comment)