liukun4515 commented on PR #3549:
URL: 
https://github.com/apache/arrow-datafusion/pull/3549#issuecomment-1254348843

   > @liukun4515 it's just the naming of the expression. the actual datatype is 
Decimal.
   > 
   > ```sql
   > ❯ select arrow_typeof(null + 1::decimal);
   > +------------------------------+
   > | arrowtypeof(NULL + Int64(1)) |
   > +------------------------------+
   > | Decimal128(38, 10)           |
   > +------------------------------+
   > 1 row in set. Query took 0.045 seconds.
   > ```
   > 
   > Naming of resulting columns after casting is kinda weird, but unrelated to 
this PR.
   > 
   > ```sql
   > ❯ select 1::decimal;
   > +--------------+
   > | Int64(1)     |
   > +--------------+
   > | 1.0000000000 |
   > +--------------+
   > 1 row in set. Query took 0.008 seconds.
   > ❯ select 1::float;
   > +----------+
   > | Int64(1) |
   > +----------+
   > | 1        |
   > +----------+
   > 1 row in set. Query took 0.005 seconds.
   > ❯ select 1::text;
   > +----------+
   > | Int64(1) |
   > +----------+
   > | 1        |
   > +----------+
   > 1 row in set. Query took 0.005 seconds.
   > ```
   
   But this make me confused.
   I think it's issue related with 
https://github.com/apache/arrow-datafusion/issues/3568
   
   cc @alamb  Do you have any comments about this.
   
   this fix looks good to me expect this  header.


-- 
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]

Reply via email to