jbossas/jboss-as — pull request #3994 by kabir (2013-02-04 13:26:45+00:00)
Title: James logging transformers

Replaces #3987 to get it to work on master-ignore. Original blurb:

Adds expression rejection testing for the logging subsystem. Also refactored to logging subsystem to put attributes with the correct resources instead of one common place.

Fixed composite operation transformer. Previously rejected operations resulted in a null transformed operation which threw an exception when attempting to add a null operation. This ignores null transformed operations. I talked to @kabir on IRC and he indicated the rejected/discarded operation should have already logged information. Ignoring seems the like the best path at this point.

The one area an issue could arise is when a lower number of results is returned when the composite operation is executed. There was a suggestion to get the failure message from the TransformedOperation, but if the operation was null we would have a mismatch in the size of the transformed operation steps and the results. Also with the test case including the in the logging susbsystem, the failure message was null as well.