Robert Bradshaw comments

Results 84 comments of


                                            Robert Bradshaw

Add a workaround for BEAM-20873 for optimized list side inputs.

Does anyone know what this test is trying to verify? It does look related, but it also looks like it might be a change detector test.

Add a workaround for BEAM-20873 for optimized list side inputs.

https://github.com/apache/beam/pull/31184

Add jinja2 as a dependency for yaml preprocessing.

R: @Polber

Rollback View.asList() side inputs optimizations.

This was resolved in https://github.com/apache/beam/pull/31163

Pass original message down through conversion for storage write api

Looks like this was approved but has conflicts that need to be resolved.

[YAML] Add the ability to pre-process yaml files with jinja2.

R: @Polber @damccorm

[YAML] Add the ability to pre-process yaml files with jinja2.

isort wants to put jinja2 after apache_beam, e.g. ``` import apache_beam as beam +import jinja2 ``` Not sure how to resolve that.

[YAML] Add the ability to pre-process yaml files with jinja2.

RowCoderTest, YamlIntegration, and Docs issues seem pre-existing.

[YAML] Add the ability to pre-process yaml files with jinja2.

A merge to master seems to have resolved the row coder issues. Running a full suite of tests again.

[YAML] Add the ability to pre-process yaml files with jinja2.

So I'm still trying to figure out why these same tests pass in isolation ( https://github.com/apache/beam/actions/runs/8722508185/job/23928542728?pr=30976 ) but fail when all tests are enabled ( https://github.com/apache/beam/actions/runs/8724953372/job/23936757826?pr=30976 ) especially when this...