[jira] Resolved: (ZOOKEEPER-926) Fork Hadoop common's test-patch.sh and modify for Zookeeper

2010-11-09 Thread Nigel Daley (JIRA)

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

Nigel Daley resolved ZOOKEEPER-926.
---

Resolution: Fixed

Committed. Resolving this issue for now.  Open new issues for further 
fixes/enhancements.

> Fork Hadoop common's test-patch.sh and modify for Zookeeper
> ---
>
> Key: ZOOKEEPER-926
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-926
> Project: Zookeeper
>  Issue Type: Improvement
>  Components: build
>Reporter: Nigel Daley
>Assignee: Nigel Daley
> Fix For: 3.4.0
>
> Attachments: ZOOKEEPER-926.patch
>
>
> Zookeeper currently uses the test-patch.sh script from the Hadoop nightly 
> dir.  This is now out of date.  I propose we just copy the updated one in 
> Hadoop common and then modify for ZK.  This will also help as ZK moves out of 
> Hadoop to it's own TLP.  

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



[jira] Commented: (ZOOKEEPER-902) Fix findbug issue in trunk "Malicious code vulnerability"

2010-11-09 Thread Nigel Daley (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12930236#action_12930236
 ] 

Nigel Daley commented on ZOOKEEPER-902:
---

The patch for this issue should also update 
src/java/test/bin/test-patch.properties to set OK_FINDBUGS_WARNINGS=0

> Fix findbug issue in trunk "Malicious code vulnerability"
> -
>
> Key: ZOOKEEPER-902
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-902
> Project: Zookeeper
>  Issue Type: Bug
>  Components: quorum, server
>Affects Versions: 3.4.0
>Reporter: Patrick Hunt
>Priority: Minor
> Fix For: 3.4.0
>
>
> https://hudson.apache.org/hudson/view/ZooKeeper/job/ZooKeeper-trunk/970/artifact/trunk/findbugs/zookeeper-findbugs-report.html#Warnings_MALICIOUS_CODE
> Malicious code vulnerability Warnings
> Code  Warning
> MSorg.apache.zookeeper.server.quorum.LeaderElection.epochGen isn't final 
> but should be

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



[jira] Updated: (ZOOKEEPER-926) Fork Hadoop common's test-patch.sh and modify for Zookeeper

2010-11-08 Thread Nigel Daley (JIRA)

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

Nigel Daley updated ZOOKEEPER-926:
--

Attachment: ZOOKEEPER-926.patch

Initial patch to replace the old test-patch file.  Further changes will be 
needed.

> Fork Hadoop common's test-patch.sh and modify for Zookeeper
> ---
>
> Key: ZOOKEEPER-926
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-926
> Project: Zookeeper
>  Issue Type: Improvement
>  Components: build
>Reporter: Nigel Daley
> Attachments: ZOOKEEPER-926.patch
>
>
> Zookeeper currently uses the test-patch.sh script from the Hadoop nightly 
> dir.  This is now out of date.  I propose we just copy the updated one in 
> Hadoop common and then modify for ZK.  This will also help as ZK moves out of 
> Hadoop to it's own TLP.  

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



[jira] Created: (ZOOKEEPER-926) Fork Hadoop common's test-patch.sh and modify for Zookeeper

2010-11-08 Thread Nigel Daley (JIRA)
Fork Hadoop common's test-patch.sh and modify for Zookeeper
---

 Key: ZOOKEEPER-926
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-926
 Project: Zookeeper
  Issue Type: Improvement
  Components: build
Reporter: Nigel Daley


Zookeeper currently uses the test-patch.sh script from the Hadoop nightly dir.  
This is now out of date.  I propose we just copy the updated one in Hadoop 
common and then modify for ZK.  This will also help as ZK moves out of Hadoop 
to it's own TLP.  

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



[jira] Created: (ZOOKEEPER-241) Build of a distro fails after clean target is run

2008-11-26 Thread Nigel Daley (JIRA)
Build of a distro fails after clean target is run
-

 Key: ZOOKEEPER-241
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-241
 Project: Zookeeper
  Issue Type: Bug
  Components: build
Affects Versions: 3.0.1
Reporter: Nigel Daley
Priority: Critical


When the ZooKeeper distro is downloaded, if the clean target is run, it deletes 
the .revision file:

{quote}
clean:
  [delete] Deleting directory /home/foo/zookeeper-3.0.1/build
  [delete] Deleting directory /home/foo/zookeeper-3.0.1/src/java/generated
  [delete] Deleting directory /home/foo/zookeeper-3.0.1/src/c/generated
  [delete] Deleting directory /home/foo/zookeeper-3.0.1/.revision
{quote}

This causes subsequent builds in the distro to fail with:

{quote}
compile-main:
   [javac] Compiling 73 source files to /home/foo/zookeeper-3.0.1/build/classes
   [javac] 
/home/foo/zookeeper-3.0.1/src/java/main/org/apache/zookeeper/Version.java:21: 
package org.apache.zookeeper.version does not exist
   [javac] public class Version implements org.apache.zookeeper.version.Info{
   [javac] ^
   ...
{quote}




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



[jira] Commented: (ZOOKEEPER-78) added a high level protocol/feature - for easy Leader Election or exclusive Write Lock creation

2008-07-24 Thread Nigel Daley (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-78?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12616591#action_12616591
 ] 

Nigel Daley commented on ZOOKEEPER-78:
--

{quote}
Fewer, simpler mechanisms generally include more people.
{quote}

+1.  Let's stick with a process for now that all contributors can use, not just 
ASF committers.

> added a high level protocol/feature - for easy Leader Election or exclusive 
> Write Lock creation
> ---
>
> Key: ZOOKEEPER-78
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-78
> Project: Zookeeper
>  Issue Type: New Feature
>  Components: java client
>Affects Versions: 3.0.0
>Reporter: james strachan
> Attachments: patch_with_including_Benjamin's_fix.patch, 
> using_zookeeper_facade.patch
>
>
> Here's a patch which adds a little WriteLock helper class for performing 
> leader elections or creating exclusive locks in some directory znode. Note 
> its an early cut; am sure we can improve it over time. The aim is to avoid 
> folks having to use the low level ZK stuff but provide a simpler high level 
> abstraction.

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



[jira] Commented: (ZOOKEEPER-83) Switch to using maven to build ZooKeeper

2008-07-24 Thread Nigel Daley (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-83?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12616583#action_12616583
 ] 

Nigel Daley commented on ZOOKEEPER-83:
--

+1 for staying with Ant and creating maven artifacts.

> Switch to using maven to build ZooKeeper
> 
>
> Key: ZOOKEEPER-83
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-83
> Project: Zookeeper
>  Issue Type: Improvement
>  Components: build
>Reporter: Hiram Chirino
> Attachments: zookeeper-mavened.tgz
>
>
> Maven is a great too for building java projects at the ASF.  It helps 
> standardize the build a bit since it's a convention oriented.
> It's dependency auto downloading would remove the need to store the 
> dependencies in svn, and it will handle many of the suggested ASF policies 
> like gpg signing of the releases and such.
> The ZooKeeper build is almost vanilla except for the jute compiler bits.  
> Things that would need to change are:
>  * re-organize the source tree a little so that it uses the maven directory 
> conventions
>  * seperate the jute bits out into seperate modules so that a maven plugin 
> can be with it
>  

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