You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by ga...@apache.org on 2008/05/03 00:41:58 UTC
svn commit: r652935 - in /incubator/pig/trunk: ./ conf/ src/org/apache/pig/
src/org/apache/pig/backend/hadoop/executionengine/ src/org/apache/pig/impl/
src/org/apache/pig/impl/util/
Author: gates
Date: Fri May 2 15:41:58 2008
New Revision: 652935
URL: http://svn.apache.org/viewvc?rev=652935&view=rev
Log:
PIG-215: Cleanup a few dangling ends left by PIG-111.
Added:
incubator/pig/trunk/conf/log4j.properties
Modified:
incubator/pig/trunk/CHANGES.txt
incubator/pig/trunk/conf/pig.properties
incubator/pig/trunk/src/org/apache/pig/Main.java
incubator/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java
incubator/pig/trunk/src/org/apache/pig/impl/PigContext.java
incubator/pig/trunk/src/org/apache/pig/impl/util/PropertiesUtil.java
Modified: incubator/pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/CHANGES.txt?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/CHANGES.txt (original)
+++ incubator/pig/trunk/CHANGES.txt Fri May 2 15:41:58 2008
@@ -267,6 +267,9 @@
PIG-176: Change bag spilling so that bags below a certain threshold are
not spilled, thus avoiding proliferation of small files (pi_song via
gates).
-
+
PIG-227: making load/store function optional in stream input/output spec
(acmurthy via olgan)
+
+ PIG-215: Cleanup a few dangling ends left by PIG-111 (pi_song via gates).
+
Added: incubator/pig/trunk/conf/log4j.properties
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/conf/log4j.properties?rev=652935&view=auto
==============================================================================
--- incubator/pig/trunk/conf/log4j.properties (added)
+++ incubator/pig/trunk/conf/log4j.properties Fri May 2 15:41:58 2008
@@ -0,0 +1,8 @@
+# ***** Set root logger level to DEBUG and its only appender to A.
+log4j.rootLogger=info, A
+
+# ***** A is set to be a ConsoleAppender.
+log4j.appender.A=org.apache.log4j.ConsoleAppender
+# ***** A uses PatternLayout.
+log4j.appender.A.layout=org.apache.log4j.PatternLayout
+log4j.appender.A.layout.ConversionPattern=%-4r [%t] %-5p %c %x - %m%n
Modified: incubator/pig/trunk/conf/pig.properties
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/conf/pig.properties?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/conf/pig.properties (original)
+++ incubator/pig/trunk/conf/pig.properties Fri May 2 15:41:58 2008
@@ -23,7 +23,7 @@
verbose=false
#exectype local|mapreduce, mapreduce is default
-exectype=local
+#exectype=mapreduce
# hod realted properties
#ssh.gateway
#hod.expect.root
Modified: incubator/pig/trunk/src/org/apache/pig/Main.java
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/src/org/apache/pig/Main.java?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/src/org/apache/pig/Main.java (original)
+++ incubator/pig/trunk/src/org/apache/pig/Main.java Fri May 2 15:41:58 2008
@@ -99,7 +99,7 @@
ExecMode mode = ExecMode.UNKNOWN;
String file = null;
- ExecType execType = ExecType.LOCAL;
+ ExecType execType = ExecType.MAPREDUCE ;
String execTypeString = properties.getProperty("exectype");
if(execTypeString!=null && execTypeString.length()>0){
execType = PigServer.parseExecType(execTypeString);
Modified: incubator/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java (original)
+++ incubator/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java Fri May 2 15:41:58 2008
@@ -135,7 +135,7 @@
}
}
- log.info("Connecting to hadoop file system at: " + properties.get(FILE_SYSTEM_LOCATION));
+ log.info("Connecting to hadoop file system at: " + (nameNode==null? LOCAL: nameNode) ) ;
ds = new HDataStorage(properties);
Configuration configuration = ConfigurationUtil.toConfiguration(properties);
Modified: incubator/pig/trunk/src/org/apache/pig/impl/PigContext.java
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/src/org/apache/pig/impl/PigContext.java?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/src/org/apache/pig/impl/PigContext.java (original)
+++ incubator/pig/trunk/src/org/apache/pig/impl/PigContext.java Fri May 2 15:41:58 2008
@@ -128,9 +128,7 @@
public PigContext(ExecType execType, Properties properties){
this.execType = execType;
- this.properties = properties;
-
- initProperties();
+ this.properties = properties;
String pigJar = JarManager.findContainingJar(Main.class);
String hadoopJar = JarManager.findContainingJar(FileSystem.class);
@@ -157,50 +155,6 @@
packageImportList.add("com.yahoo.pig.yst.sds.ULT.");
packageImportList.add("org.apache.pig.impl.builtin.");
}
-
- private void initProperties() {
- Properties fileProperties = new Properties();
-
- try{
- // first read the properties in the jar file
- InputStream pis = MapReduceLauncher.class.getClassLoader().getResourceAsStream("properties");
- if (pis != null) {
- fileProperties.load(pis);
- }
-
- //then read the properties in the home directory
- try{
- pis = new FileInputStream(System.getProperty("user.home") + "/.pigrc");
- }catch(IOException e){}
- if (pis != null) {
- fileProperties.load(pis);
- }
- }catch (IOException e){
- log.error(e);
- throw new RuntimeException(e);
- }
-
- //Now set these as system properties only if they are not already defined.
- for (Object o: fileProperties.keySet()){
- String propertyName = (String)o;
- if (log.isDebugEnabled()) {
- StringBuilder sb = new StringBuilder();
- sb.append("Found system property ");
- sb.append(propertyName);
- sb.append(" in .pigrc");
- log.debug(sb.toString());
- }
- if (System.getProperty(propertyName) == null){
- System.setProperty(propertyName, fileProperties.getProperty(propertyName));
- if (log.isDebugEnabled()) {
- StringBuilder sb = new StringBuilder();
- sb.append("Setting system property ");
- sb.append(propertyName);
- log.debug(sb.toString());
- }
- }
- }
- }
public void connect() throws ExecException {
Modified: incubator/pig/trunk/src/org/apache/pig/impl/util/PropertiesUtil.java
URL: http://svn.apache.org/viewvc/incubator/pig/trunk/src/org/apache/pig/impl/util/PropertiesUtil.java?rev=652935&r1=652934&r2=652935&view=diff
==============================================================================
--- incubator/pig/trunk/src/org/apache/pig/impl/util/PropertiesUtil.java (original)
+++ incubator/pig/trunk/src/org/apache/pig/impl/util/PropertiesUtil.java Fri May 2 15:41:58 2008
@@ -41,10 +41,7 @@
}
} catch (Exception e) {
log.error("unable to parse .pigrc :", e);
- }
-
- // merge all keys/values
- pigrcProps.putAll(properties) ;
+ }
// Now put all the entries from pigrcProps into properties, but
// only if they are not already set. pig.properties takes
@@ -56,6 +53,19 @@
}
}
+ //Now set these as system properties only if they are not already defined.
+ if (log.isDebugEnabled()) {
+ for (Object o: properties.keySet()) {
+ String propertyName = (String) o ;
+ StringBuilder sb = new StringBuilder() ;
+ sb.append("Found property ") ;
+ sb.append(propertyName) ;
+ sb.append("=") ;
+ sb.append(properties.get(propertyName).toString()) ;
+ log.debug(sb.toString()) ;
+ }
+ }
+
// For telling error fast when there are problems
ConfigurationValidator.validatePigProperties(properties) ;
}