maasvdberg/jboss-as — comment on commit 02aa4deb5d94f269921f0794540aa3bdcd8a5a39
File: controller/src/main/java/org/jboss/as/controller/AbstractAddStepHandler.java line 115
By bstansberry:

Before merging this patch, the committer needs to do a complete review of every non-subsystem subclass to see if this change is compatible. The semantic of the context.isNormalServer() call is different from context.getType() == OperationContext.Type.SERVER.

I like the change; we just need to be careful.

The change is appropriate for a subsystem handler, as no subsystem handler should be doing anything at Stage.RUNTIME if context.isNormalServer() == false.