You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by dv...@apache.org on 2012/10/02 19:06:04 UTC

svn commit: r1393036 - in /pig/trunk: CHANGES.txt src/org/apache/pig/PigServer.java

Author: dvryaboy
Date: Tue Oct  2 17:06:03 2012
New Revision: 1393036

URL: http://svn.apache.org/viewvc?rev=1393036&view=rev
Log:
PIG-2935: Catch NoSuchMethodError when StoreFuncInterface's new cleanupOnSuccess method isn't implemented.

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/PigServer.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1393036&r1=1393035&r2=1393036&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Tue Oct  2 17:06:03 2012
@@ -285,6 +285,8 @@ OPTIMIZATIONS
 
 BUG FIXES
 
+PIG-2935: Catch NoSuchMethodError when StoreFuncInterface's new cleanupOnSuccess method isn't implemented. (gates via dvryaboy)
+
 PIG-2920: e2e tests override PERL5LIB environment variable (azaroth)
 
 PIG-2917: SpillableMemoryManager memory leak for WeakReference (haitao.yao via dvryaboy)

Modified: pig/trunk/src/org/apache/pig/PigServer.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/PigServer.java?rev=1393036&r1=1393035&r2=1393036&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/PigServer.java (original)
+++ pig/trunk/src/org/apache/pig/PigServer.java Tue Oct  2 17:06:03 2012
@@ -1296,6 +1296,10 @@ public class PigServer {
                             new Job(output.getConf()));
                 } catch (IOException e) {
                     throw new ExecException(e);
+                } catch (AbstractMethodError nsme) {
+                    // Just swallow it.  This means we're running against an
+                    // older instance of a StoreFunc that doesn't implement
+                    // this method.
                 }
             }
         }