Build failed in Hudson: ZooKeeper-trunk #516

2009-11-02 Thread Apache Hudson Server
See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/516/

--
[...truncated 163860 lines...]
[junit] 2009-11-02 10:50:57,000 - INFO  
[SessionTracker:sessiontrackeri...@143] - SessionTrackerImpl exited loop!
[junit] 2009-11-02 10:50:57,005 - INFO  [main:zookee...@505] - Closing 
session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,005 - INFO  [main:clientc...@1081] - Closing 
ClientCnxn for session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,006 - INFO  
[ProcessThread:-1:preprequestproces...@384] - Processed session termination 
request for id: 0x124b484c87d
[junit] 2009-11-02 10:50:57,007 - INFO  [SyncThread:0:nioserverc...@834] - 
closing session:0x124b484c87d NIOServerCnxn: 
java.nio.channels.SocketChannel[connected local=/127.0.0.1:11225 
remote=/127.0.0.1:41017]
[junit] 2009-11-02 10:50:57,008 - INFO  
[main-SendThread(localhost:11225):clientcnxn$sendthr...@974] - Exception while 
closing send thread for session 0x124b484c87d : Read error rc = -1 
java.nio.DirectByteBuffer[pos=0 lim=4 cap=4]
[junit] 2009-11-02 10:50:57,108 - INFO  [main:clientc...@1067] - 
Disconnecting ClientCnxn for session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,108 - INFO  [main:zookee...@513] - Session: 
0x124b484c87d closed
[junit] 2009-11-02 10:50:57,108 - INFO  
[main-EventThread:clientcnxn$eventthr...@525] - EventThread shut down
[junit] 2009-11-02 10:50:57,109 - INFO  [main:clientb...@397] - tearDown 
starting
[junit] 2009-11-02 10:50:57,109 - INFO  [main:clientb...@409] - fdcount 
after test is: 42
[junit] 2009-11-02 10:50:57,109 - INFO  [main:zookee...@505] - Closing 
session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,109 - INFO  [main:clientc...@1081] - Closing 
ClientCnxn for session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,110 - INFO  [main:clientc...@1067] - 
Disconnecting ClientCnxn for session: 0x124b484c87d
[junit] 2009-11-02 10:50:57,110 - INFO  [main:zookee...@513] - Session: 
0x124b484c87d closed
[junit] 2009-11-02 10:50:57,110 - INFO  [main:clientb...@374] - STOPPING 
server
[junit] 2009-11-02 10:50:57,110 - INFO  
[NIOServerCxn.Factory:11225:nioservercnxn$fact...@236] - NIOServerCnxn factory 
exited run method
[junit] 2009-11-02 10:50:57,111 - INFO  [main:finalrequestproces...@319] - 
shutdown of request processor complete
[junit] 2009-11-02 10:50:57,111 - INFO  
[SyncThread:0:syncrequestproces...@134] - SyncRequestProcessor exited!
[junit] 2009-11-02 10:50:57,111 - INFO  
[ProcessThread:-1:preprequestproces...@119] - PrepRequestProcessor exited loop!
[junit] ensureOnly:[]
[junit] 2009-11-02 10:50:57,114 - INFO  [main:clientb...@428] - FINISHED 
testWatcherAutoResetDisabledWithGlobal
[junit] 2009-11-02 10:50:57,115 - INFO  [main:clientb...@338] - STARTING 
testWatcherAutoResetDisabledWithLocal
[junit] 2009-11-02 10:50:57,120 - INFO  [main:clientb...@367] - STARTING 
server
[junit] 2009-11-02 10:50:57,121 - INFO  [main:zookeeperser...@160] - 
Created server
[junit] 2009-11-02 10:50:57,121 - INFO  [main:nioservercnxn$fact...@122] - 
binding to port 11226
[junit] 2009-11-02 10:50:57,122 - INFO  [main:filetxnsnap...@208] - 
Snapshotting: 0
[junit] ensureOnly:[InMemoryDataTree, StandaloneServer_port]
[junit] 2009-11-02 10:50:57,124 - INFO  
[NIOServerCxn.Factory:11226:nioserverc...@699] - Processing stat command from 
/127.0.0.1:52006
[junit] 2009-11-02 10:50:57,124 - WARN  
[NIOServerCxn.Factory:11226:nioserverc...@495] - Exception causing close of 
session 0x0 due to java.io.IOException: Responded to info probe
[junit] 2009-11-02 10:50:57,124 - INFO  
[NIOServerCxn.Factory:11226:nioserverc...@834] - closing session:0x0 
NIOServerCnxn: java.nio.channels.SocketChannel[connected local=/127.0.0.1:11226 
remote=/127.0.0.1:52006]
[junit] expect:InMemoryDataTree
[junit] found:InMemoryDataTree 
org.apache.ZooKeeperService:name0=StandaloneServer_port-1,name1=InMemoryDataTree
[junit] expect:StandaloneServer_port
[junit] found:StandaloneServer_port 
org.apache.ZooKeeperService:name0=StandaloneServer_port-1
[junit] 2009-11-02 10:50:57,125 - INFO  [main:clientb...@359] - Initial 
fdcount is: 42
[junit] 2009-11-02 10:50:57,126 - INFO  [main:clientb...@363] - Client test 
setup finished
[junit] 2009-11-02 10:50:57,126 - INFO  [main:zookee...@372] - Initiating 
client connection, connectString=127.0.0.1:11226 sessionTimeout=5000 
watcher=org.apache.zookeeper.test.watchertest$mywatc...@1ca209e
[junit] 2009-11-02 10:50:57,126 - INFO  
[main-SendThread():clientcnxn$sendthr...@880] - Attempting connection to server 
/127.0.0.1:11226
[junit] 2009-11-02 10:50:57,127 - INFO  
[main-SendThread(localhost:11226):clientcnxn$sendthr...@796] - Priming 
connection to java.nio.channels.SocketChannel[connected local=/127.0.0.1:52007 
remote=localhost/127.0.0.1:11226]
[junit] 2009-11-02 

[jira] Updated: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Dudziak updated ZOOKEEPER-537:
-

Attachment: build.patch

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Dudziak updated ZOOKEEPER-537:
-

Affects Version/s: (was: 3.2.1)
   3.3.0
   Status: Patch Available  (was: Open)

This is a patch to the build.xml in trunk to add the generation of a separate 
src jar file.

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772642#action_12772642
 ] 

Hadoop QA commented on ZOOKEEPER-537:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12423846/build.patch
  against trunk revision 831486.

+1 @author.  The patch does not contain any @author tags.

-1 tests included.  The patch doesn't appear to include any new or modified 
tests.
Please justify why no tests are needed for this patch.

+1 javadoc.  The javadoc tool did not generate any warning messages.

+1 javac.  The applied patch does not increase the total number of javac 
compiler warnings.

+1 findbugs.  The patch does not introduce any new Findbugs warnings.

-1 release audit.  The applied patch generated 184 release audit warnings 
(more than the trunk's current 182 warnings).

+1 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

Test results: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/43/testReport/
Release audit warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/43/artifact/trunk/patchprocess/releaseAuditDiffWarnings.txt
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/43/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/43/console

This message is automatically generated.

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-537:
---

Status: Open  (was: Patch Available)

Thomas there are at least a couple of outstanding comments not addressed by 
this patch, specifically there
is interest to not change the existing jar file as it is currently used for 
those non-maven users.. I put forth the
idea to have a separate set of jars for bin/src/jdoc (btw, I don't see the jdoc 
jar in your patch, isn't that something
you had mentioned would be useful?)

https://issues.apache.org/jira/browse/ZOOKEEPER-537?focusedCommentId=12761052page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12761052

https://issues.apache.org/jira/browse/ZOOKEEPER-537?focusedCommentId=12763561page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12763561

Do you see an issue with what I'm suggesting?


 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Mahadev konar (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772679#action_12772679
 ] 

Mahadev konar commented on ZOOKEEPER-537:
-

thomas, 

  After reading hiram's comments on this jira, do you still think this is a 
problem and needs to get fixed? 

https://issues.apache.org/jira/browse/ZOOKEEPER-537?focusedCommentId=12767931page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_12767931

is the link to his comments.


 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772680#action_12772680
 ] 

Thomas Dudziak commented on ZOOKEEPER-537:
--

@Mahadev: Yes - see my reply above. His comparison to GWT does not apply as 
afaik you don't typically depend on the GWT builds as libraries.

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772682#action_12772682
 ] 

Thomas Dudziak commented on ZOOKEEPER-537:
--

@Patrick: I'll tweak it.

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-519) Followerhandler should close the socket if it gets an exception on a write.

2009-11-02 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-519:


Status: Patch Available  (was: Open)

 Followerhandler should close the socket if it gets an exception on a write.
 ---

 Key: ZOOKEEPER-519
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-519
 Project: Zookeeper
  Issue Type: Bug
Reporter: Mahadev konar
Assignee: Mahadev konar
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-519.patch


 We noticed this in our tests -
 {code}
 java.net.SocketException: Broken pipe
 at java.net.SocketOutputStream.socketWrite0(Native Method)
 at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:92)
 at java.net.SocketOutputStream.write(SocketOutputStream.java:136)
 at java.io.BufferedOutputStream.write(BufferedOutputStream.java:105)
 at java.io.DataOutputStream.write(DataOutputStream.java:90)
 at java.io.FilterOutputStream.write(FilterOutputStream.java:80)
 at 
 org.apache.jute.BinaryOutputArchive.writeBuffer(BinaryOutputArchive.java:122)
 at org.apache.zookeeper.server.DataNode.serialize(DataNode.java:126)
 at 
 org.apache.jute.BinaryOutputArchive.writeRecord(BinaryOutputArchive.java:126)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:878)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at org.apache.zookeeper.server.DataTree.serialize(DataTree.java:940)
 at 
 org.apache.zookeeper.server.util.SerializeUtils.serializeSnapshot(SerializeUtils.java:102)
 at 
 org.apache.zookeeper.server.ZooKeeperServer.serializeSnapshot(ZooKeeperServer.java:269)
 at 
 org.apache.zookeeper.server.quorum.FollowerHandler.run(FollowerHandler.java:263)
 {code}
 So the followerhandler got an exception while writing to the socket but the 
 follower was still waiting on the socket for a read and got a read timeout 
 after 60 seconds or so. To just make sure we handle this rightly, we should 
 close the socket at the followerhandler when we get an excpetion, so that the 
 follower immediately recognizes that its disconnected from the leader.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-519) Followerhandler should close the socket if it gets an exception on a write.

2009-11-02 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-519:


Attachment: ZOOKEEPER-519.patch

a simple fix. I dont think a test is easy to right for this.

 Followerhandler should close the socket if it gets an exception on a write.
 ---

 Key: ZOOKEEPER-519
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-519
 Project: Zookeeper
  Issue Type: Bug
Reporter: Mahadev konar
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-519.patch


 We noticed this in our tests -
 {code}
 java.net.SocketException: Broken pipe
 at java.net.SocketOutputStream.socketWrite0(Native Method)
 at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:92)
 at java.net.SocketOutputStream.write(SocketOutputStream.java:136)
 at java.io.BufferedOutputStream.write(BufferedOutputStream.java:105)
 at java.io.DataOutputStream.write(DataOutputStream.java:90)
 at java.io.FilterOutputStream.write(FilterOutputStream.java:80)
 at 
 org.apache.jute.BinaryOutputArchive.writeBuffer(BinaryOutputArchive.java:122)
 at org.apache.zookeeper.server.DataNode.serialize(DataNode.java:126)
 at 
 org.apache.jute.BinaryOutputArchive.writeRecord(BinaryOutputArchive.java:126)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:878)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at org.apache.zookeeper.server.DataTree.serialize(DataTree.java:940)
 at 
 org.apache.zookeeper.server.util.SerializeUtils.serializeSnapshot(SerializeUtils.java:102)
 at 
 org.apache.zookeeper.server.ZooKeeperServer.serializeSnapshot(ZooKeeperServer.java:269)
 at 
 org.apache.zookeeper.server.quorum.FollowerHandler.run(FollowerHandler.java:263)
 {code}
 So the followerhandler got an exception while writing to the socket but the 
 follower was still waiting on the socket for a read and got a read timeout 
 after 60 seconds or so. To just make sure we handle this rightly, we should 
 close the socket at the followerhandler when we get an excpetion, so that the 
 follower immediately recognizes that its disconnected from the leader.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-519) Followerhandler should close the socket if it gets an exception on a write.

2009-11-02 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772723#action_12772723
 ] 

Hadoop QA commented on ZOOKEEPER-519:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12423855/ZOOKEEPER-519.patch
  against trunk revision 831486.

+1 @author.  The patch does not contain any @author tags.

-1 tests included.  The patch doesn't appear to include any new or modified 
tests.
Please justify why no tests are needed for this patch.

+1 javadoc.  The javadoc tool did not generate any warning messages.

+1 javac.  The applied patch does not increase the total number of javac 
compiler warnings.

+1 findbugs.  The patch does not introduce any new Findbugs warnings.

+1 release audit.  The applied patch does not increase the total number of 
release audit warnings.

+1 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

Test results: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/44/testReport/
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/44/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/44/console

This message is automatically generated.

 Followerhandler should close the socket if it gets an exception on a write.
 ---

 Key: ZOOKEEPER-519
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-519
 Project: Zookeeper
  Issue Type: Bug
Reporter: Mahadev konar
Assignee: Mahadev konar
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-519.patch


 We noticed this in our tests -
 {code}
 java.net.SocketException: Broken pipe
 at java.net.SocketOutputStream.socketWrite0(Native Method)
 at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:92)
 at java.net.SocketOutputStream.write(SocketOutputStream.java:136)
 at java.io.BufferedOutputStream.write(BufferedOutputStream.java:105)
 at java.io.DataOutputStream.write(DataOutputStream.java:90)
 at java.io.FilterOutputStream.write(FilterOutputStream.java:80)
 at 
 org.apache.jute.BinaryOutputArchive.writeBuffer(BinaryOutputArchive.java:122)
 at org.apache.zookeeper.server.DataNode.serialize(DataNode.java:126)
 at 
 org.apache.jute.BinaryOutputArchive.writeRecord(BinaryOutputArchive.java:126)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:878)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at 
 org.apache.zookeeper.server.DataTree.serializeNode(DataTree.java:890)
 at org.apache.zookeeper.server.DataTree.serialize(DataTree.java:940)
 at 
 org.apache.zookeeper.server.util.SerializeUtils.serializeSnapshot(SerializeUtils.java:102)
 at 
 org.apache.zookeeper.server.ZooKeeperServer.serializeSnapshot(ZooKeeperServer.java:269)
 at 
 org.apache.zookeeper.server.quorum.FollowerHandler.run(FollowerHandler.java:263)
 {code}
 So the followerhandler got an exception while writing to the socket but the 
 follower was still waiting on the socket for a read and got a read timeout 
 after 60 seconds or so. To just make sure we handle this rightly, we should 
 close the socket at the followerhandler when we get an excpetion, so that the 
 follower immediately recognizes that its disconnected from the leader.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Dudziak updated ZOOKEEPER-537:
-

Attachment: (was: build.patch)

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-537) The zookeeper jar includes the java source files

2009-11-02 Thread Thomas Dudziak (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Thomas Dudziak updated ZOOKEEPER-537:
-

Attachment: build.patch

Updated patch that creates additional -bin, -src, -javadoc jars.

 The zookeeper jar includes the java source files
 

 Key: ZOOKEEPER-537
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-537
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.3.0
Reporter: Thomas Dudziak
 Fix For: 3.3.0

 Attachments: build.patch


 This is a problem if you use zookeeper as a dependency in maven because for 
 whatever reason the maven compiler plugin will pick up the java files in the 
 jar and compile them to the output directory. From there they will land in 
 the generated jar file for whatever project happens to depend on zookeeper 
 thus introducing duplicate classes (once in zookeeper.jar, once in the 
 project's artifact).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-558) server sent stats not being updated

2009-11-02 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-558?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-558:
---

Attachment: ZOOKEEPER-558.patch

This patch addresses the sent count issue (counter update was missing), 
additionally I addressed a number
of session establishment/teardown issues that I found while fixing this:

1) better logging of session est/teardown, much of this was based on user 
feedback such as ZOOKEEPER-565 ZOOKEEPER-564
Much of this was cleaning up the log text and using common terminology for the 
messages themselves
1.1) some of 1) required better error handling. A lot of the changed code was 
throwing general IOExceptions, rather than throwing
more specific exceptions allowing higher layer code to properly report the 
issue.
2) while closing a client connection the connection close response was not 
always sent to the client
3) in some cases the server would not close the connection (closeconn buffer 
was not being queued) properly. this would happen anyway when the client closed 
the connection, but the server should do the right thing regardless

4) calling close more than once just returns after the first call
5) added testableWaitForShutdown to ZooKeeper to allow tests to validate that 
the client has shutdown all threads
(this is test only)

6) reading a client sockets now reads the entire request in one shot if 
possible. previously we would re-poll the
selector after reading the length header.



 server sent stats not being updated
 -

 Key: ZOOKEEPER-558
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-558
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Critical
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-558.patch


 the server and connection sent stat is not being updated. if you run stat 
 on the client port the sent packets is much lower than it should be
 seems that sendbuffer is not updating the stats when it shortcircuits the 
 send.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-558) server sent stats not being updated

2009-11-02 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-558?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-558:
---

Status: Patch Available  (was: Open)

 server sent stats not being updated
 -

 Key: ZOOKEEPER-558
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-558
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Critical
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-558.patch


 the server and connection sent stat is not being updated. if you run stat 
 on the client port the sent packets is much lower than it should be
 seems that sendbuffer is not updating the stats when it shortcircuits the 
 send.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-558) server sent stats not being updated

2009-11-02 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772814#action_12772814
 ] 

Hadoop QA commented on ZOOKEEPER-558:
-

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12423863/ZOOKEEPER-558.patch
  against trunk revision 831486.

+1 @author.  The patch does not contain any @author tags.

+1 tests included.  The patch appears to include 15 new or modified tests.

+1 javadoc.  The javadoc tool did not generate any warning messages.

+1 javac.  The applied patch does not increase the total number of javac 
compiler warnings.

-1 findbugs.  The patch appears to introduce 1 new Findbugs warnings.

+1 release audit.  The applied patch does not increase the total number of 
release audit warnings.

+1 core tests.  The patch passed core unit tests.

+1 contrib tests.  The patch passed contrib unit tests.

Test results: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/45/testReport/
Findbugs warnings: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/45/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: 
http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/45/console

This message is automatically generated.

 server sent stats not being updated
 -

 Key: ZOOKEEPER-558
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-558
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Critical
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-558.patch


 the server and connection sent stat is not being updated. if you run stat 
 on the client port the sent packets is much lower than it should be
 seems that sendbuffer is not updating the stats when it shortcircuits the 
 send.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Created: (ZOOKEEPER-568) SyncRequestProcessor snapping too frequently - counts non-log events as log events

2009-11-02 Thread Patrick Hunt (JIRA)
SyncRequestProcessor snapping too frequently - counts non-log events as log 
events
--

 Key: ZOOKEEPER-568
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-568
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.2.1
Reporter: Patrick Hunt
 Fix For: 3.3.0


Noticed the following issues in SyncRequestProcessor

1) logCount is incremented even for non-log events (say getData)
txnlog should return indication if request was logged or not (if hdr ==null it 
returns)

also:

2) move r.nextInt below logCount++ (ie if an actual log event)
3) fix indentation after txnlog.append (for some reason has unnecessary 4 char 
indent)


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-558) server sent stats not being updated

2009-11-02 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-558?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-558:
---

Status: Open  (was: Patch Available)

 server sent stats not being updated
 -

 Key: ZOOKEEPER-558
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-558
 Project: Zookeeper
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
Priority: Critical
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-558.patch


 the server and connection sent stat is not being updated. if you run stat 
 on the client port the sent packets is much lower than it should be
 seems that sendbuffer is not updating the stats when it shortcircuits the 
 send.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-568) SyncRequestProcessor snapping too frequently - counts non-log events as log events

2009-11-02 Thread Patrick Hunt (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-568?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12772822#action_12772822
 ] 

Patrick Hunt commented on ZOOKEEPER-568:


move traceMask into isTraceEnabled check

 SyncRequestProcessor snapping too frequently - counts non-log events as log 
 events
 --

 Key: ZOOKEEPER-568
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-568
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.2.1
Reporter: Patrick Hunt
 Fix For: 3.3.0


 Noticed the following issues in SyncRequestProcessor
 1) logCount is incremented even for non-log events (say getData)
 txnlog should return indication if request was logged or not (if hdr ==null 
 it returns)
 also:
 2) move r.nextInt below logCount++ (ie if an actual log event)
 3) fix indentation after txnlog.append (for some reason has unnecessary 4 
 char indent)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.