andoni-guzman commented on PR #17530: URL: https://github.com/apache/beam/pull/17530#issuecomment-1119914944
Hi @tvalentyn . I think is ready, but I would like to another test using for example Python CoGBK Dataflow Streaming to verify that now the format is correct. Related to why in some tests publish relevant errors, in this particular case I believe it's for the kind of error, in the second one ( 5191) I saw an error that is not present in 2605 . Maybe that error invoke a log for that problem. Also , doing some research for this task, I notice that some tests use test_dataflow_runner and other dataflow_runner. Test_dataflow_runner had some logs that could help. But I added that kind of logs in both files to get more info of a failure. Yes hunting logs adds a lot of friction, maybe we can look in a future to try to add logging.error in exceptions or something else. If you have an idea of how we can improve this, I'll with the disposition to help -- 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. To unsubscribe, e-mail: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
