Ahmed Abualsaud comments

Results 43 comments of


                                            Ahmed Abualsaud

Fix failing BQ file loads test for Py37

Run Python 3.7 PostCommit

[Bug]: Beam Python failure for dataflow_exercise_metrics_pipeline_test.ExerciseMetricsPipelineTest.test_metrics_it

.take-issue

[Bug]: Beam Python failure for dataflow_exercise_metrics_pipeline_test.ExerciseMetricsPipelineTest.test_metrics_it

I'm not actively working on this

Generate external transform wrappers using a script

friendly ping @chamikaramj @robertwb

Generate external transform wrappers using a script

Thanks for looking @chamikaramj. I addressed these comments, PTAL

Generate external transform wrappers using a script

Thank you @lostluck, yes best to let it bake and get it in the next release

Generate external transform wrappers using a script

> What if instead they were in a single subdirectory (like with protos) I like this alternative more actually. I've made some changes to instead generate wrappers inside the `apache_beam.transforms._external_transforms`...

Generate external transform wrappers using a script

Successful PreCommit run: https://github.com/ahmedabu98/beam/actions/runs/7714357594/job/21026416076

Generate external transform wrappers using a script

R: @tvalentyn R: @robertwb Ready for another review!

Generate external transform wrappers using a script

Running `./gradlew generateExternalTransformWrappers` fails on my side because of this line in apache_beam.io: `from apache_beam.transforms.xlang.io import *`. It fails because the script initially cleans up the generated modules, then imports...