You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by ma...@apache.org on 2009/06/15 22:52:19 UTC
svn commit: r784970 - in /hadoop/zookeeper/trunk: ./
src/java/main/org/apache/zookeeper/server/
src/java/main/org/apache/zookeeper/server/persistence/
src/java/main/org/apache/zookeeper/server/quorum/
src/java/main/org/apache/zookeeper/server/upgrade/ ...
Author: mahadev
Date: Mon Jun 15 20:52:18 2009
New Revision: 784970
URL: http://svn.apache.org/viewvc?rev=784970&view=rev
Log:
ZOOKEEPER-443. trace logging in watch notification not wrapped with istraceneabled - inefficient (pat via mahadev)
Modified:
hadoop/zookeeper/trunk/CHANGES.txt
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/WatchManager.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerHandler.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/upgrade/UpgradeSnapShotV1.java
hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java
Modified: hadoop/zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/CHANGES.txt?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/CHANGES.txt (original)
+++ hadoop/zookeeper/trunk/CHANGES.txt Mon Jun 15 20:52:18 2009
@@ -206,6 +206,9 @@
ZOOKEEPER-437. Variety of Documentation Updates (grant via mahadev)
+ ZOOKEEPER-443. trace logging in watch notification not wrapped with
+istraceneabled - inefficient (pat via mahadev)
+
NEW FEATURES:
ZOOKEEPER-371. jdiff documentation included in build/release (giri via phunt)
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java Mon Jun 15 20:52:18 2009
@@ -512,14 +512,16 @@
updateCount(lastPrefix, -1);
updateBytes(lastPrefix, node.data == null? 0:-(node.data.length));
}
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.EVENT_DELIVERY_TRACE_MASK,
- "dataWatches.triggerWatch " + path);
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.EVENT_DELIVERY_TRACE_MASK,
- "childWatches.triggerWatch " + parentName);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.EVENT_DELIVERY_TRACE_MASK,
+ "dataWatches.triggerWatch " + path);
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.EVENT_DELIVERY_TRACE_MASK,
+ "childWatches.triggerWatch " + parentName);
+ }
Set<Watcher> processed =
- dataWatches.triggerWatch(path, EventType.NodeDeleted);
+ dataWatches.triggerWatch(path, EventType.NodeDeleted);
childWatches.triggerWatch(path, EventType.NodeDeleted, processed);
childWatches.triggerWatch(parentName.equals("")?"/":parentName, EventType.NodeChildrenChanged);
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java Mon Jun 15 20:52:18 2009
@@ -78,7 +78,9 @@
if (request.type == OpCode.ping) {
traceMask = ZooTrace.SERVER_PING_TRACE_MASK;
}
- ZooTrace.logRequest(LOG, traceMask, 'E', request, "");
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logRequest(LOG, traceMask, 'E', request, "");
+ }
ProcessTxnResult rc = null;
synchronized (zks.outstandingChanges) {
while (!zks.outstandingChanges.isEmpty()
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java Mon Jun 15 20:52:18 2009
@@ -917,10 +917,12 @@
*/
synchronized public void process(WatchedEvent event) {
ReplyHeader h = new ReplyHeader(-1, -1L, 0);
- ZooTrace.logTraceMessage(LOG, ZooTrace.EVENT_DELIVERY_TRACE_MASK,
- "Deliver event " + event + " to 0x"
- + Long.toHexString(this.sessionId)
- + " through " + this);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.EVENT_DELIVERY_TRACE_MASK,
+ "Deliver event " + event + " to 0x"
+ + Long.toHexString(this.sessionId)
+ + " through " + this);
+ }
// Convert WatchedEvent to a type that can be sent over the wire
WatcherEvent e = event.getWrapper();
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java Mon Jun 15 20:52:18 2009
@@ -97,7 +97,9 @@
if (request.type == OpCode.ping) {
traceMask = ZooTrace.CLIENT_PING_TRACE_MASK;
}
- ZooTrace.logRequest(LOG, traceMask, 'P', request, "");
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logRequest(LOG, traceMask, 'P', request, "");
+ }
if (Request.requestOfDeath == request) {
break;
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java Mon Jun 15 20:52:18 2009
@@ -142,10 +142,12 @@
}
synchronized public boolean touchSession(long sessionId, int timeout) {
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.CLIENT_PING_TRACE_MASK,
- "SessionTrackerImpl --- Touch session: 0x"
- + Long.toHexString(sessionId) + " with timeout " + timeout);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.CLIENT_PING_TRACE_MASK,
+ "SessionTrackerImpl --- Touch session: 0x"
+ + Long.toHexString(sessionId) + " with timeout " + timeout);
+ }
Session s = sessionsById.get(sessionId);
if (s == null) {
return false;
@@ -172,9 +174,11 @@
synchronized public void removeSession(long sessionId) {
Session s = sessionsById.remove(sessionId);
sessionsWithTimeout.remove(sessionId);
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "SessionTrackerImpl --- Removing session 0x"
- + Long.toHexString(sessionId));
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "SessionTrackerImpl --- Removing session 0x"
+ + Long.toHexString(sessionId));
+ }
if (s != null) {
sessionSets.get(s.tickTime).sessions.remove(s);
}
@@ -182,8 +186,10 @@
public void shutdown() {
running = false;
- ZooTrace.logTraceMessage(LOG, ZooTrace.getTextTraceLevel(),
- "Shutdown SessionTrackerImpl!");
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.getTextTraceLevel(),
+ "Shutdown SessionTrackerImpl!");
+ }
}
@@ -197,13 +203,17 @@
if (sessionsById.get(id) == null) {
Session s = new Session(id, 0);
sessionsById.put(id, s);
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "SessionTrackerImpl --- Adding session 0x"
- + Long.toHexString(id) + " " + sessionTimeout);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "SessionTrackerImpl --- Adding session 0x"
+ + Long.toHexString(id) + " " + sessionTimeout);
+ }
} else {
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "SessionTrackerImpl --- Existing session 0x"
- + Long.toHexString(id) + " " + sessionTimeout);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "SessionTrackerImpl --- Existing session 0x"
+ + Long.toHexString(id) + " " + sessionTimeout);
+ }
}
touchSession(id, sessionTimeout);
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/WatchManager.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/WatchManager.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/WatchManager.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/WatchManager.java Mon Jun 15 20:52:18 2009
@@ -88,9 +88,11 @@
synchronized (this) {
watchers = watchTable.remove(path);
if (watchers == null || watchers.isEmpty()) {
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.EVENT_DELIVERY_TRACE_MASK,
- "No watchers for " + path);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.EVENT_DELIVERY_TRACE_MASK,
+ "No watchers for " + path);
+ }
return null;
}
for (Watcher w : watchers) {
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java Mon Jun 15 20:52:18 2009
@@ -312,9 +312,11 @@
protected void killSession(long sessionId, long zxid) {
dataTree.killSession(sessionId, zxid);
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "ZooKeeperServer --- killSession: 0x"
- + Long.toHexString(sessionId));
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "ZooKeeperServer --- killSession: 0x"
+ + Long.toHexString(sessionId));
+ }
if (sessionTracker != null) {
sessionTracker.removeSession(sessionId);
}
@@ -502,9 +504,11 @@
protected void revalidateSession(ServerCnxn cnxn, long sessionId,
int sessionTimeout) throws IOException, InterruptedException {
boolean rc = sessionTracker.touchSession(sessionId, sessionTimeout);
- ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
- "Session 0x" + Long.toHexString(sessionId) +
- " is valid: " + rc);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
+ "Session 0x" + Long.toHexString(sessionId) +
+ " is valid: " + rc);
+ }
cnxn.finishSessionInit(rc);
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java Mon Jun 15 20:52:18 2009
@@ -155,19 +155,23 @@
case OpCode.createSession:
sessions.put(hdr.getClientId(),
((CreateSessionTxn) txn).getTimeOut());
- ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
- "playLog --- create session in log: "
- + Long.toHexString(hdr.getClientId())
- + " with timeout: "
- + ((CreateSessionTxn) txn).getTimeOut());
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
+ "playLog --- create session in log: "
+ + Long.toHexString(hdr.getClientId())
+ + " with timeout: "
+ + ((CreateSessionTxn) txn).getTimeOut());
+ }
// give dataTree a chance to sync its lastProcessedZxid
dt.processTxn(hdr, txn);
break;
case OpCode.closeSession:
sessions.remove(hdr.getClientId());
- ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
- "playLog --- close session in log: "
- + Long.toHexString(hdr.getClientId()));
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,ZooTrace.SESSION_TRACE_MASK,
+ "playLog --- close session in log: "
+ + Long.toHexString(hdr.getClientId()));
+ }
dt.processTxn(hdr, txn);
break;
default:
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Follower.java Mon Jun 15 20:52:18 2009
@@ -117,7 +117,9 @@
if (pp.getType() == Leader.PING) {
traceMask = ZooTrace.SERVER_PING_TRACE_MASK;
}
- ZooTrace.logQuorumPacket(LOG, traceMask, 'i', pp);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logQuorumPacket(LOG, traceMask, 'i', pp);
+ }
}
/**
@@ -291,10 +293,12 @@
cnxn.finishSessionInit(valid);
}
}
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "Session 0x"
- + Long.toHexString(sessionId)
- + " is valid: " + valid);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.SESSION_TRACE_MASK,
+ "Session 0x" + Long.toHexString(sessionId)
+ + " is valid: " + valid);
+ }
break;
case Leader.SYNC:
zk.sync();
@@ -350,10 +354,12 @@
QuorumPacket qp = new QuorumPacket(Leader.REVALIDATE, -1, baos
.toByteArray(), null);
pendingRevalidations.put(clientId, cnxn);
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.SESSION_TRACE_MASK,
- "To validate session 0x"
- + Long.toHexString(clientId));
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.SESSION_TRACE_MASK,
+ "To validate session 0x"
+ + Long.toHexString(clientId));
+ }
writePacket(qp, true);
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerHandler.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerHandler.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerHandler.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerHandler.java Mon Jun 15 20:52:18 2009
@@ -120,7 +120,9 @@
if (p.getType() == Leader.PING) {
traceMask = ZooTrace.SERVER_PING_TRACE_MASK;
}
- ZooTrace.logQuorumPacket(LOG, traceMask, 'o', p);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logQuorumPacket(LOG, traceMask, 'o', p);
+ }
oa.writeRecord(p, "packet");
} catch (IOException e) {
if (!sock.isClosed()) {
@@ -314,7 +316,9 @@
if (qp.getType() == Leader.PING) {
traceMask = ZooTrace.SERVER_PING_TRACE_MASK;
}
- ZooTrace.logQuorumPacket(LOG, traceMask, 'i', qp);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logQuorumPacket(LOG, traceMask, 'i', qp);
+ }
tickOfLastAck = leader.self.tick;
@@ -347,10 +351,12 @@
DataOutputStream dos = new DataOutputStream(bos);
dos.writeLong(id);
boolean valid = leader.zk.touch(id, to);
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.SESSION_TRACE_MASK,
- "Session 0x" + Long.toHexString(id)
- + " is valid: "+ valid);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.SESSION_TRACE_MASK,
+ "Session 0x" + Long.toHexString(id)
+ + " is valid: "+ valid);
+ }
dos.writeBoolean(valid);
qp.setData(bos.toByteArray());
queuedPackets.add(qp);
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java Mon Jun 15 20:52:18 2009
@@ -55,8 +55,10 @@
try {
while (!finished) {
Request request = queuedRequests.take();
- ZooTrace.logRequest(LOG, ZooTrace.CLIENT_REQUEST_TRACE_MASK,
- 'F', request, "");
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logRequest(LOG, ZooTrace.CLIENT_REQUEST_TRACE_MASK,
+ 'F', request, "");
+ }
if (request == Request.requestOfDeath) {
break;
}
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/upgrade/UpgradeSnapShotV1.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/upgrade/UpgradeSnapShotV1.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/upgrade/UpgradeSnapShotV1.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/upgrade/UpgradeSnapShotV1.java Mon Jun 15 20:52:18 2009
@@ -88,9 +88,11 @@
long id = ia.readLong("id");
int to = ia.readInt("timeout");
sessions.put(id, to);
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "loadData --- session in archive: " + id
- + " with timeout: " + to);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "loadData --- session in archive: " + id
+ + " with timeout: " + to);
+ }
count--;
}
oldTree.deserialize(ia, "tree");
@@ -131,21 +133,25 @@
case OpCode.createSession:
sessionsWithTimeouts.put(hdr.getClientId(),
((CreateSessionTxn) txn).getTimeOut());
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.SESSION_TRACE_MASK,
- "playLog --- create session in log: 0x"
- + Long.toHexString(hdr.getClientId())
- + " with timeout: "
- + ((CreateSessionTxn) txn).getTimeOut());
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.SESSION_TRACE_MASK,
+ "playLog --- create session in log: 0x"
+ + Long.toHexString(hdr.getClientId())
+ + " with timeout: "
+ + ((CreateSessionTxn) txn).getTimeOut());
+ }
// give dataTree a chance to sync its lastProcessedZxid
oldDataTree.processTxn(hdr, txn);
break;
case OpCode.closeSession:
sessionsWithTimeouts.remove(hdr.getClientId());
- ZooTrace.logTraceMessage(LOG,
- ZooTrace.SESSION_TRACE_MASK,
- "playLog --- close session in log: 0x"
- + Long.toHexString(hdr.getClientId()));
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG,
+ ZooTrace.SESSION_TRACE_MASK,
+ "playLog --- close session in log: 0x"
+ + Long.toHexString(hdr.getClientId()));
+ }
oldDataTree.processTxn(hdr, txn);
break;
default:
Modified: hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java
URL: http://svn.apache.org/viewvc/hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java?rev=784970&r1=784969&r2=784970&view=diff
==============================================================================
--- hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java (original)
+++ hadoop/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/util/SerializeUtils.java Mon Jun 15 20:52:18 2009
@@ -83,9 +83,11 @@
long id = ia.readLong("id");
int to = ia.readInt("timeout");
sessions.put(id, to);
- ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
- "loadData --- session in archive: " + id
- + " with timeout: " + to);
+ if (LOG.isTraceEnabled()) {
+ ZooTrace.logTraceMessage(LOG, ZooTrace.SESSION_TRACE_MASK,
+ "loadData --- session in archive: " + id
+ + " with timeout: " + to);
+ }
count--;
}
dt.deserialize(ia, "tree");