[PIG-1727] Hadoop default config override pig.properties

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8.0
• Fix Version/s: 0.8.0
• Component/s: impl
• Hadoop Flags: Reviewed
• Assignee: Richard Ding
• Reporter: Daniel Dai
• Created: 15/Nov/10 23:07
• Updated: 17/Dec/10 22:46
• Resolved: 17/Nov/10 18:50

Description

In GenericOptionsParser work, we have these lines:

GenericOptionsParser parser = new GenericOptionsParser(args);
Configuration conf = parser.getConfiguration();    

Properties properties = new Properties();
properties.putAll(ConfigurationUtil.toProperties(conf));
PropertiesUtil.loadDefaultProperties(properties);

conf contains two sources: one from hadoop default config files (core-site.xml, hdfs-site.xml, mapred-site.xml), the other is from hadoop related command line options. Override conf over pig property files is wrong for the default hadoop configuration part. We shall not bring default hadoop configuration in. The following code illustrate how to do that:

Configuration conf = new Configuration(false);
GenericOptionsParser parser = new GenericOptionsParser(conf, args);
conf = parser.getConfiguration();