[ 
https://issues.apache.org/jira/browse/BEAM-4235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17546957#comment-17546957
 ] 

Kenneth Knowles commented on BEAM-4235:
---------------------------------------

This issue has been migrated to https://github.com/apache/beam/issues/18768

> Migrate common_job_properties.groovy to use Maps for keyword argument style 
> calls instead of positional arguments
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: BEAM-4235
>                 URL: https://issues.apache.org/jira/browse/BEAM-4235
>             Project: Beam
>          Issue Type: Improvement
>          Components: testing
>            Reporter: Luke Cwik
>            Priority: P3
>              Labels: starter
>
> This follows the the style defined within 
> [build_rules.gradle|https://github.com/apache/beam/blob/684fb9f974e1b007544d6e9d57fcf99be355e99e/build_rules.gradle#L397]
>  as it is much simpler to read the following method with keyword arguments:
> setCommonJobProperties(context: context, maxWorkers: 3)
> This also simplifies the usage of methods with multiple default arguments.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to