ZOOKEEPER-4212: add committer checklist to zk-merge-pr.py

[maoling]

• Add the committer checklist to commit script to avoid The Accident Case like: PR-1597
• more details in the ZOOKEEPER-4212

[eolivelli]

I am not sure we need this stuff.
We are all responsible for what we are doing and we always wait for at least 2 binding +1.

We can always git revert.

I appreciate very much this proposal and I feel your feelings about your incident.

I would like not to commit this patch.

So -0 from my side

[maoling]

@eolivelli That's OK. If your guys don't want it, let it be my private commit script copy to give me a checklist to avoid unexpected accidents when I commit something:)

[symat]

I can only repeat myself: I really don't think you should have bad feelings about PR-1597. It wasn't a big deal at all. We all make mistakes, but as Enrico mentioned, there is nothing we couldn't fix in git :)

And also we have a great community that is continuously monitoring / testing things, which gives us a great confidence in ZooKeeper quality for the released versions.

[anmolnar]

@maoling I owe you an apology if I hurt your feelings with my e-mail. I'm sorry, I didn't mean that, I was really just kidding, but this is not fun anymore.
No need for this list in the codebase and you don't need it either. Please close this PR.