reuvenlax commented on issue #10413: [BEAM-9035] Typed options for Row Schema and Field URL: https://github.com/apache/beam/pull/10413#issuecomment-599244074 Taking another look. I'm fine with metadata being in a separate PR - it makes it easier to review. I just wanted to ensure that metadata was still in the plan. On Sun, Mar 15, 2020 at 4:24 AM Alex Van Boxel <[email protected]> wrote: > I think I addresses now all the issues in the latest fixup. > > Talking about field metadata: yes this will be another PR. I didn't want > this all in one PR. This is the groundwork to get a decent API in. I have > at least 2 related PR's (proto and avro support) waiting before tackling > removal of metadata though. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/beam/pull/10413#issuecomment-599197547>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AFAYJVLQZ6BGMQQ37KJ65QTRHS3HPANCNFSM4J4M5UTA> > . >
---------------------------------------------------------------- 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] With regards, Apache Git Services
