[GitHub] [beam] tvalentyn commented on pull request #12263: [BEAM-10492] Add missing sideinput handling to DLP transforms

2020-08-10 Thread GitBox
tvalentyn commented on pull request #12263: URL: https://github.com/apache/beam/pull/12263#issuecomment-671495214 @mwalenia 2.24.0 branch cut is in a couple of days - are you still working on this change? This is an

[GitHub] [beam] tvalentyn commented on pull request #12263: [BEAM-10492] Add missing sideinput handling to DLP transforms

2020-07-21 Thread GitBox
tvalentyn commented on pull request #12263: URL: https://github.com/apache/beam/pull/12263#issuecomment-662158727 I added the fix to the release branch. @santhh will help validate this feature on 2.23.0 RC2. I suggest we add necessary tests to this PR and merge it into master