[ 
https://issues.apache.org/jira/browse/BEAM-4083?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16444809#comment-16444809
 ] 

Kenneth Knowles commented on BEAM-4083:
---------------------------------------

Ah, that's fair. I might code up a full discriminator function (like with 
{{DoFnSignature.Parameter}} rather than doing a switch on the type constructor. 
It came up on https://github.com/apache/beam/pull/5122

> TypeName should be a proper algebraic type, and probably just called BeamType
> -----------------------------------------------------------------------------
>
>                 Key: BEAM-4083
>                 URL: https://issues.apache.org/jira/browse/BEAM-4083
>             Project: Beam
>          Issue Type: Sub-task
>          Components: sdk-java-core
>            Reporter: Kenneth Knowles
>            Priority: Major
>
> TypeName mixes atomic types and type constructors. Or, equivalently, it does 
> not distinguish by arity the type constructors. It would be best to make this 
> an ADT.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to