[
https://issues.apache.org/jira/browse/BEAM-10258?focusedWorklogId=445660&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-445660
]
ASF GitHub Bot logged work on BEAM-10258:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 15/Jun/20 05:05
Start Date: 15/Jun/20 05:05
Worklog Time Spent: 10m
Work Description: saavan-google-intern edited a comment on pull request
#12009:
URL: https://github.com/apache/beam/pull/12009#issuecomment-643904266
R: @udim
CC: @robertwb
This PR is not ready for merge, but I'm making it available so we can track
and discuss its progress.
What do you think of the current implementation? It works on some trivial
pipelines that I've run on my machine but I want to confirm the solution's
design before I debug deeper and write real tests.
Also what is the expected behavior for runtime type checking with
annotations? For example, if an input PCollection to a PTransform is annotated
with the type hint _PCollection[str]_, should the runtime type checking system
confirm that each element in the PCollection is indeed a string? Is this how
the current runtime type checking system works with decorators and inline type
hints?
Thanks!
----------------------------------------------------------------
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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 445660)
Time Spent: 0.5h (was: 20m)
> Support type hint annotations on PTransform's expand()
> ------------------------------------------------------
>
> Key: BEAM-10258
> URL: https://issues.apache.org/jira/browse/BEAM-10258
> Project: Beam
> Issue Type: Improvement
> Components: sdk-py-core
> Reporter: Saavan Nanavati
> Assignee: Saavan Nanavati
> Priority: P2
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
