Re: Unparsing based on operand type

2024-01-23 Thread Julian Hyde
gt; Also, validation does not really know about dialects. >> >> Mihai >> >> From: Tanner Clary >> Sent: Tuesday, January 23, 2024 1:00 PM >> To: dev@calcite.apache.org >> Subject: Re: Unparsing based on operand type >> >> One idea I

Re: Unparsing based on operand type

2024-01-23 Thread Tanner Clary
dev@calcite.apache.org > Subject: Re: Unparsing based on operand type > > One idea I had was: during validation when the return type for a call is > derived, you could have a property in `SqlBasicCall` called something like > "unparseOperator". You could have some logic

Re: Unparsing based on operand type

2024-01-23 Thread Mihai Budiu
Node to unparsing won't > make this problem much simpler, although there you can at least count on > the representation being properly typed. > > Mihai > > From: Tanner Clary > Sent: Tuesday, January 23, 2024 12:32 PM > To: dev@calcit

Re: Unparsing based on operand type

2024-01-23 Thread Tanner Clary
to unparsing won't > make this problem much simpler, although there you can at least count on > the representation being properly typed. > > Mihai > > From: Tanner Clary > Sent: Tuesday, January 23, 2024 12:32 PM > To: dev@calcite.apache.org

Re: Unparsing based on operand type

2024-01-23 Thread Mihai Budiu
oblem much simpler, although there you can at least count on the representation being properly typed. Mihai From: Tanner Clary Sent: Tuesday, January 23, 2024 12:32 PM To: dev@calcite.apache.org Subject: Unparsing based on operand type Hey Calcite Devs, I'm

Unparsing based on operand type

2024-01-23 Thread Tanner Clary
Hey Calcite Devs, I'm working on CALCITE-6220 and the summary is that we should unparse MIN/MAX differently for Postgres/Redshift depending on its operand type (if the type is BOOLEAN, it should be rewritten as BOOL_AND/OR). This problem seems