JMSQueueAppender: posting logged message several times

Question

package com.citco.banking.nxg.logger;
* Copyright 1999-2005 The Apache Software Foundation.
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
* http://www.apache.org/licenses/LICENSE-2.0
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
import org.apache.log4j.AppenderSkeleton;
import org.apache.log4j.spi.LoggingEvent;
import org.apache.log4j.spi.ErrorHandler;
import org.apache.log4j.spi.ErrorCode;
import org.apache.log4j.helpers.LogLog;
import java.util.Date;
import java.util.Hashtable;
import java.util.Properties;
import javax.jms.*;
import javax.naming.InitialContext;
import javax.naming.Context;
import javax.naming.NameNotFoundException;
import javax.naming.NamingException;
* A Simple JMS (P2P) Queue Appender.
* @author Ceki G�lc�
* @author Jamie Tsao
public class JMSQueueAppender extends AppenderSkeleton {
protected QueueConnection queueConnection;
protected QueueSession queueSession;
protected QueueSender queueSender;
protected Queue queue;
String initialContextFactory;
String providerUrl;
String queueBindingName;
String queueConnectionFactoryBindingName;
public
JMSQueueAppender() {
* The InitialContextFactory option takes a string value.
* Its value, along with the ProviderUrl option will be used
* to get the InitialContext.
public void setInitialContextFactory(String initialContextFactory) {
this.initialContextFactory = initialContextFactory;
* Returns the value of the InitialContextFactory option.
public String getInitialContextFactory() {
return initialContextFactory;
* The ProviderUrl option takes a string value.
* Its value, along with the InitialContextFactory option will be used
* to get the InitialContext.
public void setProviderUrl(String providerUrl) {
this.providerUrl = providerUrl;
* Returns the value of the ProviderUrl option.
public String getProviderUrl() {
return providerUrl;
* The QueueConnectionFactoryBindingName option takes a
* string value. Its value will be used to lookup the appropriate
* QueueConnectionFactory from the JNDI context.
public void setQueueConnectionFactoryBindingName(String queueConnectionFactoryBindingName) {
this.queueConnectionFactoryBindingName = queueConnectionFactoryBindingName;
* Returns the value of the QueueConnectionFactoryBindingName option.
public String getQueueConnectionFactoryBindingName() {
return queueConnectionFactoryBindingName;
* The QueueBindingName option takes a
* string value. Its value will be used to lookup the appropriate
* destination Queue from the JNDI context.
public void setQueueBindingName(String queueBindingName) {
this.queueBindingName = queueBindingName;
Returns the value of the QueueBindingName option.
public String getQueueBindingName() {
return queueBindingName;
* Overriding this method to activate the options for this class
* i.e. Looking up the Connection factory ...
public void activateOptions() {
QueueConnectionFactory queueConnectionFactory;
try {
System.out.println("inside activateOptions");
Context ctx = getInitialContext();
queueConnectionFactory = (QueueConnectionFactory) ctx.lookup(queueConnectionFactoryBindingName);
queueConnection = queueConnectionFactory.createQueueConnection();
queueSession = queueConnection.createQueueSession(false,
Session.AUTO_ACKNOWLEDGE);
Queue queue = (Queue) ctx.lookup(queueBindingName);
queueSender = queueSession.createSender(queue);
queueConnection.start();
ctx.close();
} catch(Exception e) {
errorHandler.error("Error while activating options for appender named ["+name+
"].", e, ErrorCode.GENERIC_FAILURE);
protected InitialContext getInitialContext() throws NamingException {
try {
Hashtable ht = new Hashtable();
//Populate property hashtable with data to retrieve the context.
ht.put(Context.INITIAL_CONTEXT_FACTORY, initialContextFactory);
ht.put(Context.PROVIDER_URL, providerUrl);
return (new InitialContext(ht));
} catch (NamingException ne) {
LogLog.error("Could not get initial context with ["+initialContextFactory + "] and [" + providerUrl + "].");
throw ne;
protected boolean checkEntryConditions() {
String fail = null;
if(this.queueConnection == null) {
fail = "No QueueConnection";
} else if(this.queueSession == null) {
fail = "No QueueSession";
} else if(this.queueSender == null) {
fail = "No QueueSender";
if(fail != null) {
errorHandler.error(fail +" for JMSQueueAppender named ["+name+"].");
return false;
} else {
return true;
* Close this JMSQueueAppender. Closing releases all resources used by the
* appender. A closed appender cannot be re-opened.
public synchronized // avoid concurrent append and close operations
void close() {
if(this.closed)
return;
LogLog.debug("Closing appender ["+name+"].");
this.closed = true;
try {
if(queueSession != null)
queueSession.close();
if(queueConnection != null)
queueConnection.close();
} catch(Exception e) {
LogLog.error("Error while closing JMSQueueAppender ["+name+"].", e);
// Help garbage collection
queueSender = null;
queueSession = null;
queueConnection = null;
* This method called by {@link AppenderSkeleton#doAppend} method to
* do most of the real appending work. The LoggingEvent will be
* be wrapped in an ObjectMessage to be put on the JMS queue.
public void append(LoggingEvent event) {
if(!checkEntryConditions()) {
System.out.println("inside checkEntryCondions()");
return;
try {
MapMessage msg = queueSession.createMapMessage();
//ObjectMessage msg = queueSession.createObjectMessage();
//msg.setString("LOGDate",new Date(event.timeStamp).toString());
msg.setJMSTimestamp(event.timeStamp);
//msg.setString("Logger",event.getLoggerName());
msg.setString("Priority",event.getLevel().toString());
msg.setString("Loc_ClassName",event.getLocationInformation().getClassName());
msg.setString("Loc_MethodName",event.getLocationInformation().getMethodName());
msg.setString("Loc_FileName",event.getLocationInformation().getFileName());
msg.setString("Loc_LineNumber",event.getLocationInformation().getLineNumber());
msg.setString("Msg", event.getMessage() == null ? "null" : event.getMessage().toString());
//msg.setObject(event);
queueSender.send(msg);
} catch(Exception e) {
errorHandler.error("Could not send message in JMSQueueAppender ["+name+"].", e,
ErrorCode.GENERIC_FAILURE);
public boolean requiresLayout() {
return false;
Above is the code for the JMSQueueAppender that I'm using for appending in the queue.
But when it's running, it is sending the message 2 to 5 times, dipending upon the LEVEL. If you're using setlevel(LEVEL.INFO) and calling the logger as logger.INFO(string str), it should give only one result but, it is posting the message twice.
Actually it's execeuting the append method inside JMSQueueAppender twice. why this method is called twice? Can anyone tell me the solution for that.

Answer 1

Some people find it annoying when web pages forward to another page automatically. You can turn off that message if you like.
orange Firefox button ''or'' Tools menu > Options > Advanced
On the "General" mini-tab, ''uncheck'' "Warn me when web site try to redirect or reload the page"