tvalentyn

Results 174 comments of tvalentyn

hmm, now two more ML suites are failing. Will check if this a new error first.

We should also fix enable onnx in dependency compat test suite: https://github.com/apache/beam/issues/25796. Beam supports protobuf3, so we should still be able to test onnx even if it doesn't support protobuf4.

> put a lower bound on docker-py for the fix version. nit: `docker`, not `docker-py`

Thanks @baeminbo for a detailed repro. Eventually, we plan to swtich to cloudpickle pickler, which doesn't require saving the main session. Structuring a pipeline as a package is the best...

Looks like we merged this with beam_PreCommit_RAT (Run RAT PreCommit) failing, and it is still broken at master @ahmedabu98 - could you PTAL? Thanks!

``` > Task :rat Files with unapproved licenses: - /runner/_work/beam/beam/sdks/python/apache_beam/transforms/managed_iceberg_it_test.py ```

Closed and reopened to rerun the tests

https://github.com/apache/beam/actions/workflows/beam_PostCommit_XVR_Direct.yml?query=branch%3Amaster looks permared

Which of the errors above are a regression in 2.59.0 (if any) ?