[ 
https://issues.apache.org/jira/browse/BEAM-7629?focusedWorklogId=302298&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-302298
 ]

ASF GitHub Bot logged work on BEAM-7629:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 27/Aug/19 18:31
            Start Date: 27/Aug/19 18:31
    Worklog Time Spent: 10m 
      Work Description: youngoli commented on issue #8936: [BEAM-7629] Go SDK 
additional Validation for DoFns (1st impl) (DO NOT MERGE)
URL: https://github.com/apache/beam/pull/8936#issuecomment-525427769
 
 
   Going to get back to this, don't close it yet.
 
----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 302298)
    Time Spent: 50m  (was: 40m)

> Improve DoFn method validation in core/graph/fn.go
> --------------------------------------------------
>
>                 Key: BEAM-7629
>                 URL: https://issues.apache.org/jira/browse/BEAM-7629
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-go
>            Reporter: Daniel Oliveira
>            Assignee: Daniel Oliveira
>            Priority: Major
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Various improvements can be made to validating the signatures and type usages 
> in DoFns. Some things that should probably be checked:
>  * Check that StartBundle and FinishBundle contain any emit parameters and 
> side inputs present in ProcessElement
>  * Check that any side inputs/emits have correctly matching types between 
> Start/FinishBundle and ProcessElement
>  * Check that parameters and return values for the various methods are valid 
> (for ex. Teardown/Setup should have no params)



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to