Issue 3127: change-merged event is not emitted when note-db is enabled
Status:  ChangeUnderReview
Owner: ----
Project Member Reported by david.pu...@sonymobile.com, Jan 26, 2015
*****************************************************************
*****                                                       *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!!  *****
*****                                                       *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD,  *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
*****                                                       *****
*****   THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS     *****
*****                                                       *****
*****************************************************************

Affected Version: master

What steps will reproduce the problem?
1. Run the site with note-db enabled
2. Merge a change
3.

What is the expected output? What do you see instead?

Expect to received a ChangeMergedEvent.  No event is received.


Please provide any additional information below.

Running in the debugger shows that the event is not fired because submitter is null.

MergeOp.java @ line 976:

    if (submitter != null) {
      try {
        hooks.doChangeMergedHook(c,
            accountCache.get(submitter.getAccountId()).getAccount(),
            merged, db, mergeResultRev);
      } catch (OrmException ex) {
        logError("Cannot run hook for submitted patch set " + c.getId(), ex);
      }
    }

Nov 9, 2015
Project Member #1 david.pu...@sonymobile.com
https://gerrit-review.googlesource.com/#/c/72235/
Status: ChangeUnderReview