On Fri, Jun 12, 2020 at 4:12 PM Brian Hulette <bhule...@google.com> wrote:
> > are unknown fields propagated through if the user only reads/modifies a > row? > I'm not sure I understand this question. Are you asking about handling > schema changes? > The wire format includes the number of fields in the schema, specifically > so that we can detect when the schema changes. This is restricted to added > or removed fields at the end of the schema. i.e. if we receive an element > that says it has N more fields than the schema this coder was created with > we assume the pipeline was updated with a schema that drops the last N > fields and ignore the extra fields. Similarly if we receive an element with > N fewer fields than we expect we'll just fill the last N fields with nulls. > This logic is implemented in Python [1] and Java [2], but it's not > exercised since no runners actually support pipeline update with schema > changes. > > > how does it work in a pipeline update scenario (downgrade / upgrade)? > It's a standard coder with a defined spec [3] and tests in > standard_coders.yaml [4] (although we could certainly use more coverage > there) so I think pipeline update should work fine, unless I'm missing > something. > The big question is whether the pipeline update will be rejected due to the Coder having "changed." > Brian > > [1] > https://github.com/apache/beam/blob/master/sdks/python/apache_beam/coders/row_coder.py#L177-L189 > [2] > https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/RowCoderGenerator.java#L341-L356 > [3] > https://github.com/apache/beam/blob/master/model/pipeline/src/main/proto/beam_runner_api.proto#L833-L864 > [4] > https://github.com/apache/beam/blob/master/model/fn-execution/src/main/resources/org/apache/beam/model/fnexecution/v1/standard_coders.yaml#L344-L364 > > On Fri, Jun 12, 2020 at 3:32 PM Luke Cwik <lc...@google.com> wrote: > >> +Boyuan Zhang <boyu...@google.com> >> >> On Fri, Jun 12, 2020 at 3:32 PM Luke Cwik <lc...@google.com> wrote: >> >>> What is the update / compat story around schemas? >>> * are unknown fields propagated through if the user only reads/modifies >>> a row? >>> * how does it work in a pipeline update scenario (downgrade / upgrade)? >>> >>> Boyuan has been working on a Kafka via SDF source and have been trying >>> to figure out which interchange format to use for the "source descriptors" >>> that feed into the SDF. Some obvious choices are json, avro, proto, and >>> Beam schemas all with their caveats. >>> >>> On Fri, Jun 12, 2020 at 1:32 PM Brian Hulette <bhule...@google.com> >>> wrote: >>> >>>> Thanks! I see there are jiras for SpannerIO and JdbcIO as part of that. >>>> Are you planning on using row coder for them? >>>> If so I want to make sure you're aware of >>>> https://s.apache.org/beam-schema-io (sent to the dev list last week >>>> [1]). +Scott Lukas <slu...@google.com> will be working on building out >>>> the ideas there this summer. His work could be useful for making these IOs >>>> cross-language (and you would get a mapping to SQL out of it without much >>>> more effort). >>>> >>>> Brian >>>> >>>> [1] >>>> https://lists.apache.org/thread.html/rc1695025d41c5dc38cdf7bc32bea0e7421379b1c543c2d82f69aa179%40%3Cdev.beam.apache.org%3E >>>> >>>> On Tue, Jun 2, 2020 at 9:30 AM Piotr Szuberski < >>>> piotr.szuber...@polidea.com> wrote: >>>> >>>>> Sure, I'll do that >>>>> >>>>> On 2020/05/28 17:54:49, Chamikara Jayalath <chamik...@google.com> >>>>> wrote: >>>>> > Great. Thanks for working on this. Can you please add these tasks >>>>> and JIRAs >>>>> > to the cross-language transforms roadmap under "Connector/transform >>>>> > support". >>>>> > https://beam.apache.org/roadmap/connectors-multi-sdk/ >>>>> > >>>>> > Happy to help if you run into any issues during this task. >>>>> > >>>>> > <https://beam.apache.org/roadmap/connectors-multi-sdk/>Thanks, >>>>> > Cham >>>>> > >>>>> > On Thu, May 28, 2020 at 9:59 AM Piotr Szuberski < >>>>> piotr.szuber...@polidea.com> >>>>> > wrote: >>>>> > >>>>> > > I added to Jira task of creating cross-language wrappers for Java >>>>> IOs. It >>>>> > > will soon be in progress. >>>>> > > >>>>> > >>>>> >>>>