You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by kr...@apache.org on 2007/01/24 09:11:38 UTC

svn commit: r499304 - /db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java

Author: kristwaa
Date: Wed Jan 24 00:11:37 2007
New Revision: 499304

URL: http://svn.apache.org/viewvc?view=rev&rev=499304
Log:
DERBY-2103: Merged from trunk.

Modified:
    db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java

Modified: db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java
URL: http://svn.apache.org/viewvc/db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java?view=diff&rev=499304&r1=499303&r2=499304
==============================================================================
--- db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java (original)
+++ db/derby/code/branches/10.2/java/engine/org/apache/derby/impl/sql/compile/ParserImpl.java Wed Jan 24 00:11:37 2007
@@ -156,6 +156,15 @@
 		}
 		catch (TokenMgrError e)
 		{
+			// Derby - 2103.
+			// When the exception occurs cachedParser may live with
+			// some flags set inappropriately that may cause Exception
+			// in the subsequent compilation. This seems to be a javacc bug.
+			// Issue Javacc-152 has been raised.
+			// As a workaround, the cachedParser object is cleared to ensure
+			// that the exception does not have any side effect.
+			// TODO : Remove the following line if javacc-152 is fixed.
+			cachedParser = null;
 		    throw StandardException.newException(SQLState.LANG_LEXICAL_ERROR, e.getMessage());
 		}
 	}