Ramesh N has posted comments on this change. Change subject: <WIP>engine: support for external event handlers ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/25270/2/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/events/ExternalEventHandler.java File backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/events/ExternalEventHandler.java: Line 5: public interface ExternalEventHandler { Line 6: Line 7: void handleEvent(AuditLog event); Line 8: Line 9: boolean canHandleEvent(AuditLog event); > What is the usage of that? I see it is returning true in all implementation It can be used to validate the event parameters. Currently it is not used fully as still it is WIP. For example it can verify that volumeID is passed. It will return false if the volume ID is not passed in eventParameters list. -- To view, visit http://gerrit.ovirt.org/25270 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4b9c8638fbc8794cc196134093af9ff7e7cb4876 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Ramesh N <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
