Ahmed Abualsaud

Results 124 comments of Ahmed Abualsaud

R: @robertwb R: @chamikaramj CC: @lostluck (if this affects Go at all) CC: @Polber (if any more changes are needed for YAML)

Thanks for the insight @lostluck! From a cursory search I see [xlang/bigtableio/bigtableio.go](https://github.com/apache/beam/blob/e764fc9c17dbc575d4a16b5e6454f4d38ef752be/sdks/go/pkg/beam/io/xlang/bigtableio/bigtable.go#L102) is currently the only SchemaTransform use-case in the Go SDK, is that right?

R: @lostluck > manual field name override of the given snake case name. let me know if the last commit does what you meant here

Thanks @Polber for catching those, can you PTAL again?

R: @chamikaramj R: @damccorm

Opening again because the workflow is broken for the past few days. Since #33555 got in, it's been failing with the following: ``` Caused by: java.lang.NoClassDefFoundError: org/apache/beam/vendor/grpc/v1p60p1/io/grpc/Channel ``` I wonder...

R: @kennknowles CC: @BhupiSindhwani CC: @emkornfield

Thanks for taking a look @kennknowles. I see what you mean about including dependencies only if they're mandatory. These dependency changes aren't necessary for this PR so I reverted them...

R: @gaolemeng R: @agrawal-siddharth CC: @reuvenlax CC: @damccorm