jfarr commented on pull request #11318: URL: https://github.com/apache/beam/pull/11318#issuecomment-664038458
> @jfarr Btw, do you think we could implement `KinesisIO.Write` with AWS SDK v2 in effective way (since KPL is still on SDL v1)? I'm sure it's possible but it seems like it would basically entail porting the KPL to AWS v2 ourselves, so I'm not sure if it's worth the effort. What would be the value gained, besides having the reader and writer together in the same class again? In my opinion I've always felt that the way Beam IOs combine readers and writers in the same class violates single-responsibility principle, with predictable results such as this. ---------------------------------------------------------------- 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]
