Hi all, I was checking https://spark-prs.appspot.com/users who has PRs more
then 10.

viirya <https://spark-prs.appspot.com/users/viirya> 13
mgaido91 <https://spark-prs.appspot.com/users/mgaido91> 12
wangyum <https://spark-prs.appspot.com/users/wangyum> 12
maropu <https://spark-prs.appspot.com/users/maropu> 11
rdblue <https://spark-prs.appspot.com/users/rdblue> 10
kiszk <https://spark-prs.appspot.com/users/kiszk> 10Please update PRs. Few
thoughts on what to update given my rough skimming:
  - If there is something blocking your PR, probably it's good to leave the
PR closed and make a progress on the things that blocks your PRs.
  - In the above case, don't forget to take an action to the JIRA. If you
are hesitant about what you should do against the JIRA, cc committers to
take an action to the JIRA
  - Take a look for your change again and please kindly reconsider if each
PR is worth enough going ahead, for example, check if the amount of change
is worth.
  - Double check if there's any hole.
  - Check if the test and build passed and update.
  - If everything is ready and up to date and you are sure on the proposed
change, please ping related committers or reviewers.
  - Please give some summary (for example, what's the next action needed,
what's blocking, what's a potential downside or risk of the proposed
change, etc.) for the current status in
      the comment or PR description if possible. That could make the review
easier.

These are mostly based up on https://spark.apache.org/contributing.html
Please kindly consider rereading it and in particular  "Closing Your Pull
Request / JIRA" too.

Reply via email to