You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by tu...@apache.org on 2012/06/21 23:06:54 UTC

svn commit: r1352687 - in /incubator/oozie/trunk: client/src/main/java/org/apache/oozie/client/OozieClient.java release-log.txt

Author: tucu
Date: Thu Jun 21 21:06:54 2012
New Revision: 1352687

URL: http://svn.apache.org/viewvc?rev=1352687&view=rev
Log:
OOZIE_846 OozieClient iterates over Properties using Hashtable method (lars_francke via tucu)

Modified:
    incubator/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java
    incubator/oozie/trunk/release-log.txt

Modified: incubator/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java
URL: http://svn.apache.org/viewvc/incubator/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java?rev=1352687&r1=1352686&r2=1352687&view=diff
==============================================================================
--- incubator/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java (original)
+++ incubator/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java Thu Jun 21 21:06:54 2012
@@ -29,7 +29,6 @@ import java.net.URL;
 import java.net.URLEncoder;
 import java.util.ArrayList;
 import java.util.Collections;
-import java.util.Enumeration;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
@@ -464,16 +463,8 @@ public class OozieClient {
             Element conf = doc.createElement("configuration");
             doc.appendChild(conf);
             conf.appendChild(doc.createTextNode("\n"));
-            for (Enumeration e = props.keys(); e.hasMoreElements();) {
-                String name = (String) e.nextElement();
-                Object object = props.get(name);
-                String value;
-                if (object instanceof String) {
-                    value = (String) object;
-                }
-                else {
-                    continue;
-                }
+            for (String name : props.stringPropertyNames()) { // Properties whose key or value is not of type String are omitted.
+                String value = props.getProperty(name);
                 Element propNode = doc.createElement("property");
                 conf.appendChild(propNode);
 

Modified: incubator/oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/incubator/oozie/trunk/release-log.txt?rev=1352687&r1=1352686&r2=1352687&view=diff
==============================================================================
--- incubator/oozie/trunk/release-log.txt (original)
+++ incubator/oozie/trunk/release-log.txt Thu Jun 21 21:06:54 2012
@@ -1,5 +1,6 @@
 -- Oozie 3.3.0 release (trunk - unreleased)
 
+OOZIE_846 OozieClient iterates over Properties using Hashtable method (lars_francke via tucu)
 OOZIE-875 Support for multiple job-xml elements in extension action nodes (rkanter via tucu)
 OOZIE-880 oozie.job.acl is not working (virag)
 OOZIE-12  Support for multiple job-xml elements in action nodes (rkanter via tucu)