apache/beam

Migrate common_job_properties.groovy to use Maps for keyword argument style calls instead of positional arguments

Open

#18,768 创建于 2022年6月3日

在 GitHub 查看
 (0 评论) (0 反应) (0 负责人)Java (4,097 fork)batch import
P3good first issueimprovementtests

仓库指标

Star
 (7,313 star)
PR 合并指标
 (平均合并 6天) (30 天内合并 243 个 PR)

描述

This follows the the style defined within build_rules.gradle 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.

Imported from Jira BEAM-4235. Original Jira may contain additional context. Reported by: lcwik.

贡献者指南

研究方向
检查 build rules.gradle 中第 397 行附近的风格，然后通过将位置参数转换为基于映射的调用来将关键字参数风格应用于 common job properties.groovy。
技术栈
groovy
领域
build system
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1 小时以内
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GroovyGradle
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60