[PIG-1955] PhysicalOperator has a member variable (non-static) Log object that is non-transient, this causes serialization errors

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8.0, 0.9.0
• Fix Version/s: 0.9.0
• Component/s: impl
• Patch Info: Patch Available
• Hadoop Flags: Reviewed
• Release Note: fix Serialzeable bug in PhysicalOperator, mark log as transient
• Assignee: Woody Anderson
• Reporter: Woody Anderson
• Created: 31/Mar/11 18:26
• Updated: 23/Apr/12 17:13
• Resolved: 06/Apr/11 17:11

Description

I found this while trying to write unit tests. Creating a local PigServer to test my LoadFunc caused a serialization of the PhysicalOperator class, which failed due to:
..
Caused by: java.io.NotSerializableException: org.apache.commons.logging.impl.Log4JCategoryLog
..

this is easily fixed by adding the transient keyword to the definition of log.

e.g.

on trunk:
private final transient Log log = LogFactory.getLog(getClass());
on the 0.8 tag:
private transient Log log = LogFactory.getLog(getClass());