Author: ggregory
Date: Wed May 14 05:17:57 2014
New Revision: 1594457

URL: http://svn.apache.org/r1594457
Log:
Refactor constants.

Added:
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/JSONConstants.java
      - copied, changed from r1594453, 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JSONConstants.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/XMLConstants.java
      - copied, changed from r1594453, 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLConstants.java
Removed:
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JSONConstants.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLConstants.java
Modified:
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/LogEventMixIn.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/MarkerMixIn.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JacksonFactory.java
    
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLLayout.java

Copied: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/JSONConstants.java
 (from r1594453, 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JSONConstants.java)
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/JSONConstants.java?p2=logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/JSONConstants.java&p1=logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JSONConstants.java&r1=1594453&r2=1594457&rev=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JSONConstants.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/JSONConstants.java
 Wed May 14 05:17:57 2014
@@ -14,13 +14,18 @@
  * See the license for the specific language governing permissions and
  * limitations under the license.
  */
-package org.apache.logging.log4j.core.layout;
+package org.apache.logging.log4j.core.jackson;
 
 /**
  * Keeps constants separate from any class that may depend on third party jars.
  */
 public final class JSONConstants {
-    public static final String PROP_SOURCE = "source";
-    public static final String PROP_CONTEXT_MAP = "contextMap";
-    public static final String PROP_CONTEXT_STACK = "contextStack";
+    public static final String ELT_CAUSE = "cause";
+    public static final String ELT_CONTEXT_MAP = "contextMap";
+    public static final String ELT_CONTEXT_STACK = "contextStack";
+    public static final String ELT_MARKER = "marker";
+    public static final String ELT_PARENTS = "parents";
+    public static final String ELT_SOURCE = "source";
+    public static final String ELT_SUPPRESSED = "suppressed";
+    public static final String ELT_THROWN = "thrown";
 }
\ No newline at end of file

Modified: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/LogEventMixIn.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/LogEventMixIn.java?rev=1594457&r1=1594456&r2=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/LogEventMixIn.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/LogEventMixIn.java
 Wed May 14 05:17:57 2014
@@ -23,8 +23,6 @@ import org.apache.logging.log4j.Marker;
 import org.apache.logging.log4j.ThreadContext.ContextStack;
 import org.apache.logging.log4j.core.LogEvent;
 import org.apache.logging.log4j.core.impl.ThrowableProxy;
-import org.apache.logging.log4j.core.layout.JSONConstants;
-import org.apache.logging.log4j.core.layout.XMLConstants;
 import org.apache.logging.log4j.message.Message;
 
 import com.fasterxml.jackson.annotation.JsonFilter;
@@ -37,24 +35,24 @@ import com.fasterxml.jackson.databind.an
 import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
 import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
 
-@JsonRootName("Event")
-@JacksonXmlRootElement(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Event")
+@JsonRootName(XMLConstants.ELT_EVENT)
+@JacksonXmlRootElement(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_EVENT)
 @JsonFilter("org.apache.logging.log4j.core.impl.Log4jLogEvent")
-@JsonPropertyOrder({ "timeMillis", "threadName", "level", "loggerName", 
"marker", "message", "thrown", XMLConstants.TAG_CONTEXT_MAP,
-        JSONConstants.PROP_CONTEXT_STACK, "loggerFQCN", "Source", "endOfBatch" 
})
+@JsonPropertyOrder({ "timeMillis", "threadName", "level", "loggerName", 
"marker", "message", "thrown", XMLConstants.ELT_CONTEXT_MAP,
+        JSONConstants.ELT_CONTEXT_STACK, "loggerFQCN", "Source", "endOfBatch" 
})
 abstract class LogEventMixIn implements LogEvent {
 
     private static final long serialVersionUID = 1L;
 
-    @JsonProperty(JSONConstants.PROP_CONTEXT_MAP)
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.TAG_CONTEXT_MAP)
+    @JsonProperty(JSONConstants.ELT_CONTEXT_MAP)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_CONTEXT_MAP)
     @JsonSerialize(using = ListOfMapEntrySerializer.class)
     @JsonDeserialize(using = ListOfMapEntryDeserializer.class)
     @Override
     public abstract Map<String, String> getContextMap();
 
-    @JsonProperty(JSONConstants.PROP_CONTEXT_STACK)
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.TAG_CONTEXT_STACK)
+    @JsonProperty(JSONConstants.ELT_CONTEXT_STACK)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_CONTEXT_STACK)
     @Override
     public abstract ContextStack getContextStack();
 
@@ -73,20 +71,20 @@ abstract class LogEventMixIn implements 
     @Override
     public abstract String getLoggerName();
 
-    @JsonProperty("marker")
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Marker")
+    @JsonProperty(JSONConstants.ELT_MARKER)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_MARKER)
     @Override
     public abstract Marker getMarker();
 
     @JsonSerialize(using = MessageSerializer.class)
     @JsonDeserialize(using = SimpleMessageDeserializer.class)
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Message")
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_MESSAGE)
     @Override
     public abstract Message getMessage();
 
-    @JsonProperty(JSONConstants.PROP_SOURCE)
+    @JsonProperty(JSONConstants.ELT_SOURCE)
     @JsonDeserialize(using = Log4jStackTraceElementDeserializer.class)
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.TAG_SOURCE)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_SOURCE)
     @Override
     public abstract StackTraceElement getSource();
 
@@ -99,8 +97,8 @@ abstract class LogEventMixIn implements 
     @Override
     public abstract Throwable getThrown();
 
-    @JsonProperty("thrown")
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Thrown")
+    @JsonProperty(JSONConstants.ELT_THROWN)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_THROWN)
     @Override
     public abstract ThrowableProxy getThrownProxy();
 

Modified: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/MarkerMixIn.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/MarkerMixIn.java?rev=1594457&r1=1594456&r2=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/MarkerMixIn.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/MarkerMixIn.java
 Wed May 14 05:17:57 2014
@@ -17,7 +17,6 @@
 package org.apache.logging.log4j.core.jackson;
 
 import org.apache.logging.log4j.Marker;
-import org.apache.logging.log4j.core.layout.XMLConstants;
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonIgnore;
@@ -56,8 +55,8 @@ abstract class MarkerMixIn implements Ma
     public abstract Marker getParent();
 
     @Override
-    @JsonProperty("parents")
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Parents")
+    @JsonProperty(JSONConstants.ELT_PARENTS)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_PARENTS)
     public abstract Marker[] getParents();
 
 }

Modified: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java?rev=1594457&r1=1594456&r2=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/ThrowableProxyMixIn.java
 Wed May 14 05:17:57 2014
@@ -18,7 +18,6 @@ package org.apache.logging.log4j.core.ja
 
 import org.apache.logging.log4j.core.impl.ExtendedStackTraceElement;
 import org.apache.logging.log4j.core.impl.ThrowableProxy;
-import org.apache.logging.log4j.core.layout.XMLConstants;
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonProperty;
@@ -29,8 +28,8 @@ import com.fasterxml.jackson.dataformat.
  */
 abstract class ThrowableProxyMixIn {
 
-    @JsonProperty("cause")
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Cause")
+    @JsonProperty(JSONConstants.ELT_CAUSE)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_CAUSE)
     private ThrowableProxyMixIn causeProxy;
 
     @JsonProperty
@@ -38,7 +37,7 @@ abstract class ThrowableProxyMixIn {
     private int commonElementCount;
 
     @JsonProperty
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"ExtendedStackTrace")
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_EXTENDED_STACK_TRACE)
     private ExtendedStackTraceElement[] extendedStackTrace;
 
     @JsonProperty
@@ -65,8 +64,8 @@ abstract class ThrowableProxyMixIn {
     @JsonIgnore
     public abstract StackTraceElement[] getStackTrace();
 
-    @JsonProperty("suppressed")
-    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
"Suppressed")
+    @JsonProperty(JSONConstants.ELT_SUPPRESSED)
+    @JacksonXmlProperty(namespace = XMLConstants.XML_NAMESPACE, localName = 
XMLConstants.ELT_SUPPRESSED)
     public abstract ThrowableProxy[] getSuppressedProxies();
 
     @JsonIgnore

Copied: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/XMLConstants.java
 (from r1594453, 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLConstants.java)
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/XMLConstants.java?p2=logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/XMLConstants.java&p1=logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLConstants.java&r1=1594453&r2=1594457&rev=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLConstants.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/jackson/XMLConstants.java
 Wed May 14 05:17:57 2014
@@ -14,14 +14,22 @@
  * See the license for the specific language governing permissions and
  * limitations under the license.
  */
-package org.apache.logging.log4j.core.layout;
+package org.apache.logging.log4j.core.jackson;
 
 /**
  * Keeps constants separate from any class that may depend on third party jars.
  */
 public final class XMLConstants {
-    public static final String TAG_SOURCE = "Source";
-    public static final String TAG_CONTEXT_MAP = "ContextMap";
-    public static final String TAG_CONTEXT_STACK = "ContextStack";
+    public static final String ELT_CAUSE = "Cause";
+    public static final String ELT_CONTEXT_MAP = "ContextMap";
+    public static final String ELT_CONTEXT_STACK = "ContextStack";
+    public static final String ELT_EVENT = "Event";
+    public static final String ELT_EXTENDED_STACK_TRACE = "ExtendedStackTrace";
+    public static final String ELT_MARKER = "Marker";
+    public static final String ELT_MESSAGE = "Message";
+    public static final String ELT_PARENTS = "Parents";
+    public static final String ELT_SOURCE = "Source";
+    public static final String ELT_SUPPRESSED = "Suppressed";
+    public static final String ELT_THROWN = "Thrown";
     public static final String XML_NAMESPACE = 
"http://logging.apache.org/log4j/2.0/events";;
 }
\ No newline at end of file

Modified: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JacksonFactory.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JacksonFactory.java?rev=1594457&r1=1594456&r2=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JacksonFactory.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/JacksonFactory.java
 Wed May 14 05:17:57 2014
@@ -20,8 +20,10 @@ import java.util.HashSet;
 import java.util.Set;
 
 import org.apache.logging.log4j.core.impl.Log4jLogEvent;
+import org.apache.logging.log4j.core.jackson.JSONConstants;
 import org.apache.logging.log4j.core.jackson.Log4jJsonObjectMapper;
 import org.apache.logging.log4j.core.jackson.Log4jXmlObjectMapper;
+import org.apache.logging.log4j.core.jackson.XMLConstants;
 
 import com.fasterxml.jackson.core.PrettyPrinter;
 import com.fasterxml.jackson.core.util.DefaultPrettyPrinter;
@@ -38,12 +40,12 @@ abstract class JacksonFactory {
 
         @Override
         protected String getPropertNameForContextMap() {
-            return JSONConstants.PROP_CONTEXT_MAP;
+            return JSONConstants.ELT_CONTEXT_MAP;
         }
 
         @Override
         protected String getPropertNameForSource() {
-            return JSONConstants.PROP_SOURCE;
+            return JSONConstants.ELT_SOURCE;
         }
 
         @Override
@@ -66,12 +68,12 @@ abstract class JacksonFactory {
 
         @Override
         protected String getPropertNameForContextMap() {
-            return XMLConstants.TAG_CONTEXT_MAP;
+            return XMLConstants.ELT_CONTEXT_MAP;
         }
 
         @Override
         protected String getPropertNameForSource() {
-            return XMLConstants.TAG_SOURCE;
+            return XMLConstants.ELT_SOURCE;
         }
 
         @Override

Modified: 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLLayout.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLLayout.java?rev=1594457&r1=1594456&r2=1594457&view=diff
==============================================================================
--- 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLLayout.java
 (original)
+++ 
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/layout/XMLLayout.java
 Wed May 14 05:17:57 2014
@@ -23,6 +23,7 @@ import java.util.Map;
 import org.apache.logging.log4j.core.config.plugins.Plugin;
 import org.apache.logging.log4j.core.config.plugins.PluginAttribute;
 import org.apache.logging.log4j.core.config.plugins.PluginFactory;
+import org.apache.logging.log4j.core.jackson.XMLConstants;
 import org.apache.logging.log4j.core.util.Charsets;
 
 /**


Reply via email to