Github user shaoxuan-wang commented on the issue:

    https://github.com/apache/flink/pull/3579
  
    @twalthr, I saw you have removed the CHAR type when you merge this. Just 
want to clarify. In FLINK-3916, in method typeInfoToSqlTypeName, you have added 
    ```
          case CHAR_TYPE_INFO | CHAR_VALUE_TYPE_INFO =>
            throw TableException("Character type is not supported.")
    ```
    TableAPI is open for user to pass CHAR type data, 
    ```
    val data = List(                                    
      (1L, 1, 1d, 1f, new BigDecimal("1"), 'a', "Hi")
    ...
    .toTable(tEnv, 'long, 'int, 'double, 'float, 'bigdec, 'string) 
    ...
    select('char.count)
    ```
    I am curious the reason why we do not support CHAR type explicitly.
    
    Also, as I suggested early, we'd better to have all build-in aggregates to 
support the retraction method. We should add string type for 
Max/MinWithRetractAggFunction. What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to