damccorm commented on code in PR #32915:
URL: https://github.com/apache/beam/pull/32915#discussion_r1825047308
##########
website/www/site/data/capability_matrix.yaml:
##########
@@ -393,7 +393,7 @@ capability-matrix:
- class: dataflow
l1: "Partially"
l2: non-merging windows
- l3: State is supported for non-merging windows. SetState and
MapState are not yet supported.
+ l3: State is supported for non-merging windows. The MapState,
SetState, and MultimapState state types are supported in the following
scenarios: Java pipelines that don't use Streaming Engine; Java pipelines that
use Streaming Engine and version 2.58.0 or later of the Java SDK. SetState,
MapState, and MultimapState are not supported for pipelines that use Runner v2.
Review Comment:
```suggestion
l3: "State is supported for non-merging windows. The MapState,
SetState, and MultimapState state types are supported in the following
scenarios: Java pipelines that don't use Streaming Engine; Java pipelines that
use Streaming Engine and version 2.58.0 or later of the Java SDK. SetState,
MapState, and MultimapState are not supported for pipelines that use Runner v2."
```
I think the `:` is causing precommits to fail and quoting should fix it
--
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]
