You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Luke Cwik (JIRA)" <ji...@apache.org> on 2018/05/04 17:44:00 UTC

[jira] [Created] (BEAM-4235) Migrate common_job_properties.groovy to use Maps for keyword argument style calls instead of positional arguments

Luke Cwik created BEAM-4235:
-------------------------------

             Summary: 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


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
(v7.6.3#76005)