Github user snuyanzin commented on the issue:

    https://github.com/apache/flink/pull/6032
  
    @twalthr thank you for your review.
    To be honest thought about such approach indeed 
org.apache.flink.api.common.typeinfo.BasicTypeInfo#toString should be changed 
at least to fix case from here 
https://issues.apache.org/jira/browse/FLINK-9341?focusedCommentId=16475693&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16475693
 .
    So I have 2 questions:
    1. is it ok to change public api behavior here (toString is public)?
    2. Should we go through all successors of 
org.apache.flink.api.common.typeinfo.TypeInformation and if its toString looks 
like clazz.getSimpleName than change it similar to 
org.apache.flink.api.java.typeutils.GenericTypeInfo#toString?
    
    could you please clarify it and than I could do required corrections


---

Reply via email to