You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by na...@apache.org on 2012/07/17 06:54:41 UTC
svn commit: r1362364 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java
Author: namit
Date: Tue Jul 17 04:54:41 2012
New Revision: 1362364
URL: http://svn.apache.org/viewvc?rev=1362364&view=rev
Log:
HIVE-3248 lack of semi-colon in .q file leads to missing the next statement
(Edward Capriolo via namit)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java?rev=1362364&r1=1362363&r2=1362364&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java Tue Jul 17 04:54:41 2012
@@ -104,6 +104,9 @@ public class SetProcessor implements Com
private CommandProcessorResponse setVariable(String varname, String varvalue){
SessionState ss = SessionState.get();
+ if (varvalue.contains("\n")){
+ ss.err.println("Warning: Value had a \\n character in it.");
+ }
if (varname.startsWith(SetProcessor.ENV_PREFIX)){
ss.err.println("env:* variables can not be set.");
return new CommandProcessorResponse(1);