Hi Dawid,
thanks for proposing this FLIP. It nicely completes the serialization
effort started with FLIP-393. I know naming is difficult, but could we
shorten and simplify the signatures further?
My suggestion would be:
OperationSerializationContext -> SqlSerializationContext
ExpressionSerializationContext -> SqlSerializationContext
Given that we want to expose this via EnvironmentSettings, it should
have an easy to understand name. `Operations` are rather internal
classes which is why `Sql` might be a better fit.
Regards,
Timo
On 08.01.25 16:27, Dawid Wysakowicz wrote:
Hi all,
I'd like to start a discussion on adding a context to Operation and
Expression SQL serialization stack to allow for serializing more expression
that require custom handling, such as inline functions. Please see[1] for
details.
Looking forward to hearing from you.
Best,
Dawid
[1] https://cwiki.apache.org/confluence/x/NIogF
