kennknowles commented on pull request #13726:
URL: https://github.com/apache/beam/pull/13726#issuecomment-760502151
I like the idea of the Beam connector knowing as little as possible. Maybe
we should accept exactly the same type? I imagine that to make this easy there
might be some `? extends Object` sort of stuff and potentially "unchecked"
casting needed? I don't have context on why the difference.
If it is actually a new ("experimental") IO that we just forgot to label as
such, I am personally still OK with breaking changes. This would be a classic
case of what that is for - refining the API until it is something we believe
can work "forever".
----------------------------------------------------------------
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]
