zhengruifeng commented on code in PR #57818: URL: https://github.com/apache/spark/pull/57818#discussion_r3822039164
########## AGENTS.md: ########## @@ -202,6 +202,8 @@ It lists `master` and the latest major's release branches the commit reached (e. PR title format is `[SPARK-xxxx][COMPONENT] Title`. Draft, WIP, MINOR, and TRIVIAL PRs may omit the JIRA ID. The component tag is derived from the JIRA component name: take the last word and uppercase it (e.g. `Project Infra` → `[INFRA]`, `Spark Core` → `[CORE]`, `Structured Streaming` → `[STREAMING]`, `SQL` → `[SQL]`). +Use `[FOLLOWUP]` only for small PRs that directly modify or correct unreleased earlier PRs. For separately planned work, non-trivial changes, or work outside the earlier JIRA's scope, create a separate JIRA ticket for each PR and use the normal title format without `[FOLLOWUP]`. Review Comment: I checked `dev/merge_spark_pr.py`: `[FOLLOW-UP]` is accepted as an alias, but the canonical tag is `[FOLLOWUP]`; the title parser normalizes `[FOLLOW-UP]` to `[FOLLOWUP]`. So I think keeping `[FOLLOWUP]` here matches the merge script behavior. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
