Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/21991#discussion_r208047588
--- Diff: dev/merge_spark_pr.py ---
@@ -142,6 +142,9 @@ def merge_pr(pr_num, target_ref, title, body,
pr_repo_desc):
distinct_authors[0])
if primary_author == "":
primary_author = distinct_authors[0]
+ else:
+ # When primary author is specified manually, put it at the head of
author list.
+ distinct_authors.insert(0, primary_author)
--- End diff --
@viirya good catch. When we put it at the head of the author list, do we
need to dedupe it again?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
