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

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

                Author: ASF GitHub Bot
            Created on: 29/May/19 11:36
            Start Date: 29/May/19 11:36
    Worklog Time Spent: 10m 
      Work Description: alexvanboxel commented on issue #8690: [BEAM-7274] 
Implement the Protobuf schema provider
URL: https://github.com/apache/beam/pull/8690#issuecomment-496899229
 
 
   Addition info, so I tried to get the logic using the FieldValueGetter with 
**partial** success, see experiment: 
https://github.com/alexvanboxel/beam/commit/d1a8e16a6d8a48b899ed769e4d0837937c31977c
   
   My main issue is with RowWithGetters private `get(Map/List)Value` values 
that assume they have enough information to construct the embedded complex data 
structures. I could get rid of my ProtoRow if I could let the handling of the 
List/Row/Map to it's dedicated `FieldValueGetter`
 
----------------------------------------------------------------
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: 250048)
    Time Spent: 50m  (was: 40m)

> Protobuf Beam Schema support
> ----------------------------
>
>                 Key: BEAM-7274
>                 URL: https://issues.apache.org/jira/browse/BEAM-7274
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Alex Van Boxel
>            Assignee: Alex Van Boxel
>            Priority: Minor
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Add support for the new Beam Schema to the Protobuf extension.



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

Reply via email to