lukecwik commented on a change in pull request #11165: [BEAM-9340] Populate
requirements for Java.
URL: https://github.com/apache/beam/pull/11165#discussion_r395306659
##########
File path:
runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/PTransformTranslation.java
##########
@@ -124,6 +124,15 @@
public static final String
SPLITTABLE_PROCESS_SIZED_ELEMENTS_AND_RESTRICTIONS_URN =
"beam:transform:sdf_process_sized_element_and_restrictions:v1";
+ public static final String REQUIRES_STATEFUL_PROCESSING_URN =
+ getUrn(RunnerApi.StandardRequirements.Enum.REQUIRES_STATEFUL_PROCESSING);
Review comment:
Please define string constants and then validate that they match in the
static initializer. This allows for the string constants to be used in
switch/case statements since they are compile time constants.
I have pr/11162 which duplicates this effort, which ever goes in second
needs to get updated.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
