[GitHub] [beam] rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream

2020-03-04 Thread GitBox
rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream URL: https://github.com/apache/beam/pull/10497#issuecomment-594701194 > This sounds like interesting voodoo. Could you eventually bring and explanation of this and motivation to the ML Sure thing!

[GitHub] [beam] rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream

2020-03-03 Thread GitBox
rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream URL: https://github.com/apache/beam/pull/10497#issuecomment-594169195 Taking a look. For some reason it's failing with a pickling error: PicklingError: Can't pickle : it's not found as

[GitHub] [beam] rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream

2020-02-26 Thread GitBox
rohdesamuel commented on issue #10497: [BEAM-8335] Add the ReverseTestStream URL: https://github.com/apache/beam/pull/10497#issuecomment-591758520 Hey Robert, I think this is good to review. Please review for content, I haven't had the time to add better comments yet (also still needs