nbali commented on PR #16909: URL: https://github.com/apache/beam/pull/16909#issuecomment-1098595429
There are missing features in both direction. The SDF already has more functionality - just not everything. So unless someone actually bothers to do it for both way for every property this tester will never be obsolete. Not to mention that the unit test introduced for this is a good reminder that a functionality has to be implemented twice for full support. The only occasion I could think of when this should be removed is when the SDF completely replaces the legacy read. Will that ever happen? -- 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]
