prashantksharma commented on issue #361:
URL:
https://github.com/apache/datafusion-comet/issues/361#issuecomment-2094569664
@andygrove
Thank you so much for your feedback:
> On the Rust side you will need a match statement to convert the protobuf
i32 to the Rust enum (0 -> legacy, 1 -> try, 2 -> ansi). Perhaps take a look at
how we handle one of the other enums defined in the proto and follow the same
pattern?
I will make updates based on my understanding of the above comment by you.
And, Thank you for your suggesting creation of draft PR, after making
necessary updates, I will do that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
