Github user texasmichelle commented on a diff in the pull request:

    https://github.com/apache/spark/pull/5149#discussion_r27092698
  
    --- Diff: dev/merge_spark_pr.py ---
    @@ -285,6 +286,56 @@ def resolve_jira_issues(title, merge_branches, 
comment):
         for jira_id in jira_ids:
             resolve_jira_issue(merge_branches, comment, jira_id)
     
    +def standardize_jira_ref(text):
    +    # Standardize the [MODULE] SPARK-XXXXX prefix
    --- End diff --
    
    Great idea, Josh. Check out the latest diff and see if it's what you had in 
mind. 
    I like what you have in spark-pr-dashboard. Would it make more sense to 
reference your code from the merge script instead? It's a bit different & would 
require a bit of finagling, but I hate to have similar code duplicated in 
different places.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to