gemini-code-assist[bot] commented on code in PR #38497:
URL: https://github.com/apache/beam/pull/38497#discussion_r3238462310
##########
runners/portability/java/build.gradle:
##########
@@ -214,6 +214,10 @@ def createUlrValidatesRunnerTask = { name,
environmentType, dockerImageTask = ""
// TODO(https://github.com/apache/beam/issues/31231)
excludeTestsMatching
'org.apache.beam.sdk.transforms.RedistributeTest.testRedistributePreservesMetadata'
+ // Failed with "KeyError: 'beam:coder:schema:v1'". New schema coder urn
is not yet supported in runners other than dataflow
Review Comment:
To improve maintainability and traceability, it is recommended to include a
`TODO` with a link to the tracking issue when excluding tests (sickbaying).
This follows the existing pattern in this file (e.g., line 214) and ensures the
exclusion is revisited once the underlying issue is resolved.
```
// TODO(https://github.com/apache/beam/issues/33859): Failed with
"KeyError: 'beam:coder:schema:v1'". New schema coder urn is not yet supported
in runners other than dataflow
```
--
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]
