You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by xu...@apache.org on 2013/05/25 00:13:00 UTC
svn commit: r1486235 - in /pig/trunk:
src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java
test/org/apache/pig/parser/TestErrorHandling.java
Author: xuefu
Date: Fri May 24 22:12:59 2013
New Revision: 1486235
URL: http://svn.apache.org/r1486235
Log:
PIG-3335 TestErrorHandling.tesNegative7 fails on MR2
Modified:
pig/trunk/src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java
pig/trunk/test/org/apache/pig/parser/TestErrorHandling.java
Modified: pig/trunk/src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java?rev=1486235&r1=1486234&r2=1486235&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java (original)
+++ pig/trunk/src/org/apache/pig/newplan/logical/rules/InputOutputFileValidator.java Fri May 24 22:12:59 2013
@@ -66,7 +66,7 @@ public class InputOutputFileValidator {
}
dummyJob = new Job(ConfigurationUtil.toConfiguration(pigCtx.getProperties()));
sf.setStoreLocation(outLoc, dummyJob);
- } catch (IOException ioe) {
+ } catch (Exception ioe) {
if(ioe instanceof PigException){
errCode = ((PigException)ioe).getErrorCode();
}
Modified: pig/trunk/test/org/apache/pig/parser/TestErrorHandling.java
URL: http://svn.apache.org/viewvc/pig/trunk/test/org/apache/pig/parser/TestErrorHandling.java?rev=1486235&r1=1486234&r2=1486235&view=diff
==============================================================================
--- pig/trunk/test/org/apache/pig/parser/TestErrorHandling.java (original)
+++ pig/trunk/test/org/apache/pig/parser/TestErrorHandling.java Fri May 24 22:12:59 2013
@@ -136,6 +136,7 @@ public class TestErrorHandling {
} catch(FrontendException ex) {
System.out.println( ex.getCause().getMessage() );
Assert.assertTrue( ex.getCause().getMessage().contains( "line 2, column 0" ) );
+ Assert.assertTrue( ex.getCause().getMessage().contains( "No FileSystem for scheme: fs2you" ) );
return;
}
Assert.fail( "Testcase should fail" );