[
https://issues.apache.org/jira/browse/BEAM-4076?focusedWorklogId=125553&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-125553
]
ASF GitHub Bot logged work on BEAM-4076:
----------------------------------------
Author: ASF GitHub Bot
Created on: 20/Jul/18 16:25
Start Date: 20/Jul/18 16:25
Worklog Time Spent: 10m
Work Description: reuvenlax commented on issue #5955: [BEAM-4076] Enable
schemas for more runners
URL: https://github.com/apache/beam/pull/5955#issuecomment-406652348
@echauchot thanks for the review! Some comments:
1. Schemas are plumbed through the NewDoFn implementation, which is why we
needed the input and output coders. Schemas are implemented as a special type
of coder (SchemaCoder), so NewDoFn needs those coders to infer schemas. Other
transforms such as CoGroupByKey are (currently) untouched by schemas. There
will be a new schema-specific Join transform (allowing you to join by specific
field names, etc.), but I don't currently plan on changing CoGroupByKey itself.
2. I'll take a look at the apex runner failures, and rebase.
3. I'll look at variable names. I tried to rename a number of them to be
less confusing, but might've missed a few :)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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: 125553)
Time Spent: 14.5h (was: 14h 20m)
> Schema followups
> ----------------
>
> Key: BEAM-4076
> URL: https://issues.apache.org/jira/browse/BEAM-4076
> Project: Beam
> Issue Type: Improvement
> Components: beam-model, dsl-sql, sdk-java-core
> Reporter: Kenneth Knowles
> Priority: Major
> Time Spent: 14.5h
> Remaining Estimate: 0h
>
> This umbrella bug contains subtasks with followups for Beam schemas, which
> were moved from SQL to the core Java SDK and made to be type-name-based
> rather than coder based.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
