reuvenlax commented on PR #38313: URL: https://github.com/apache/beam/pull/38313#issuecomment-5648373881
@chamikaramj @damccorm @ahmedabu98 this causes StorageApiSinkSchemaUpdateIT.testAtLeastOnceDynamicDestinationsWithAutoSchemaUpdate to fail. AFAICT from debugging, the tests fail because we don't get a schema-update response back from BigQuery before the test finishes. I'll follow up with BQ team to ensure there is no bug here, but truth is that this response was never guaranteed to be synchronous (there's caching in the write API, etc.); this test was already potentially flaky, and I suspect that this change just pushed it into failing more consistently. My current inclination is to just force connection pooling off for this test, and merge the PR. What do you all think? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
