[GitHub] zookeeper issue #726: ZOOKEEPER-3205: o.a.jute test cases

2018-11-28 Thread akkumar
Github user akkumar commented on the issue: https://github.com/apache/zookeeper/pull/726 @eolivelli - import static added. Let me know if this looks ok ! ---

[GitHub] zookeeper pull request #726: ZOOKEEPER-3205: o.a.jute test cases

2018-11-27 Thread akkumar
Github user akkumar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/726#discussion_r236957794 --- Diff: zookeeper-jute/src/test/java/org/apache/jute/BinaryInputArchiveTest.java --- @@ -41,4 +42,136 @@ public void testReadStringCheckLength

[GitHub] zookeeper issue #726: ZOOKEEPER-3205: o.a.jute test cases

2018-11-27 Thread akkumar
Github user akkumar commented on the issue: https://github.com/apache/zookeeper/pull/726 Ok - in the logs I see this. Any suggestion what needs to be done to get this sorted out ? ``` [exec] [exec] [exec] Error: No value specified

[GitHub] zookeeper issue #726: ZOOKEEPER-3205: o.a.jute test cases

2018-11-27 Thread akkumar
Github user akkumar commented on the issue: https://github.com/apache/zookeeper/pull/726 The test cases seem to run with no failures. It is not obvious what caused the jenkins failure ? Any suggestions where to look for ? ---

[GitHub] zookeeper pull request #726: ZOOKEEPER-3205: o.a.jute test cases

2018-11-26 Thread akkumar
GitHub user akkumar opened a pull request: https://github.com/apache/zookeeper/pull/726 ZOOKEEPER-3205: o.a.jute test cases Adding some test cases for o.a.jute BinaryInputArchive . Issue ZOOKEEPER-3205 contains the patch as well. You can merge this pull request into a Git