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