nbali commented on PR #16888: URL: https://github.com/apache/beam/pull/16888#issuecomment-1103048448
tl;dr: v1 runner replaces sdf read with legacy read silently which is a _very bad_ user experience - it should be decided if sdf configuration with v1 runner should fail during the graph creation, or if it should be allowed and let the user handle the upcoming issues - this middle state is a pita - I went with one direction, but I don't care which is the final one, I'm just waiting for someone to say it out EXPLICITLY from the beam team which one to do... the current middle one is bad -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
