Merge pull request #964 from zhujunxxxxx/master

增加mysql存储过程创建时参数的可选类型

src/main/java/com/alibaba/druid/sql/dialect/mysql/ast/clause/MySqlParameter.java

@@ -32,8 +32,17 @@ public class MySqlParameter extends MySqlObjectImpl {
     private SQLExpr     name;
     private SQLDataType dataType;
     private SQLExpr     defaultValue;
+    private ParameterType paramType;
 
-    public SQLExpr getDefaultValue() {
+    public ParameterType getParamType() {
+		return paramType;
+	}
+
+	public void setParamType(ParameterType paramType) {
+		this.paramType = paramType;
+	}
+
+	public SQLExpr getDefaultValue() {
         return defaultValue;
     }
 
@@ -66,5 +75,15 @@ public class MySqlParameter extends MySqlObjectImpl {
         }
         visitor.endVisit(this);
     }
+    /**
+     * mysql procedure parameter type
+     */
+    public static enum ParameterType
+    {
+    	DEFAULT,
+    	IN,//in
+    	OUT,//out
+    	INOUT//inout
+    }
 
 }

src/main/java/com/alibaba/druid/sql/dialect/mysql/parser/MySqlStatementParser.java

@@ -73,6 +73,7 @@ import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlIterateStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlLeaveStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlLoopStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlParameter;
+import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlParameter.ParameterType;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlRepeatStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlSelectIntoStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlWhileStatement;
@@ -2785,7 +2786,28 @@ public class MySqlStatementParser extends SQLStatementParser {
 				parameter.setDataType(dataType);
 
 				parameter.setDefaultValue(new SQLQueryExpr(select));
+				
+			} else if (lexer.token() == Token.IN || lexer.token() == Token.OUT || lexer.token() == Token.INOUT) {
+				
+				if(lexer.token()==Token.IN)
+				{
+					parameter.setParamType(ParameterType.IN);
+				}
+				else if(lexer.token()==Token.OUT)
+				{
+					parameter.setParamType(ParameterType.OUT);
+				}
+				else if(lexer.token()==Token.INOUT)
+				{
+					parameter.setParamType(ParameterType.INOUT);
+				}
+				lexer.nextToken();
+
+				parameter.setName(this.exprParser.name());
+
+				parameter.setDataType(this.exprParser.parseDataType());
 			} else {
+				parameter.setParamType(ParameterType.DEFAULT);//default parameter type is in
 				parameter.setName(this.exprParser.name());
 				parameter.setDataType(this.exprParser.parseDataType());
 

src/main/java/com/alibaba/druid/sql/dialect/mysql/visitor/MySqlOutputVisitor.java

@@ -59,6 +59,7 @@ import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlIterateStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlLeaveStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlLoopStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlParameter;
+import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlParameter.ParameterType;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlRepeatStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlSelectIntoStatement;
 import com.alibaba.druid.sql.dialect.mysql.ast.clause.MySqlWhileStatement;
@@ -3277,6 +3278,19 @@ public class MySqlOutputVisitor extends SQLASTOutputVisitor implements MySqlASTV
             decrementIndent();
 
         } else {
+        	
+        	if(x.getParamType()==ParameterType.IN)
+        	{
+        		print("IN ");
+        	}
+        	else if(x.getParamType()==ParameterType.OUT)
+        	{
+        		print("OUT ");
+        	}
+        	else if(x.getParamType()==ParameterType.INOUT)
+        	{
+        		print("INOUT ");
+        	}
             x.getName().accept(this);
             print(" ");
 

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/ast/stmt/SQLServerExecStatement.java

@@ -1,8 +1,3 @@
-package com.alibaba.druid.sql.dialect.sqlserver.ast.stmt;
-
-import java.util.ArrayList;
-import java.util.List;
-
 /*
  * Copyright 1999-2101 Alibaba Group Holding Ltd.
  *
@@ -18,6 +13,11 @@ import java.util.List;
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+package com.alibaba.druid.sql.dialect.sqlserver.ast.stmt;
+
+import java.util.ArrayList;
+import java.util.List;
+
 import com.alibaba.druid.sql.ast.SQLExpr;
 import com.alibaba.druid.sql.ast.SQLName;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.SQLServerObjectImpl;
@@ -28,7 +28,7 @@ public class SQLServerExecStatement extends SQLServerObjectImpl implements SQLSe
 
     private SQLName       returnStatus;
     private SQLName       moduleName;
-    private List<SQLExpr> parameters = new ArrayList<SQLExpr>();
+    private List<SQLServerParameter> parameters = new ArrayList<SQLServerParameter>();
     private String        dbType;
 
     public SQLName getModuleName() {
@@ -39,7 +39,7 @@ public class SQLServerExecStatement extends SQLServerObjectImpl implements SQLSe
         this.moduleName = moduleName;
     }
 
-    public List<SQLExpr> getParameters() {
+    public List<SQLServerParameter> getParameters() {
         return parameters;
     }
 
@@ -67,4 +67,37 @@ public class SQLServerExecStatement extends SQLServerObjectImpl implements SQLSe
     public void setDbType(String dbType) {
         this.dbType = dbType;
     }
+    
+    /**
+     * 
+     * @Description: SQLServer execute Parameter statement
+     * @author zz email:455910092@qq.com
+     * @date 2015-9-20
+     * @version V1.0
+     */
+    public static class SQLServerParameter extends SQLServerObjectImpl
+    {
+    	private SQLExpr expr;
+    	private boolean type;//sql server 支持参数只有input 和 output 两种
+		public SQLExpr getExpr() {
+			return expr;
+		}
+		public void setExpr(SQLExpr expr) {
+			this.expr = expr;
+		}
+		public boolean getType() {
+			return type;
+		}
+		public void setType(boolean type) {
+			this.type = type;
+		}
+		@Override
+		public void accept0(SQLServerASTVisitor visitor) {
+			if (visitor.visit(this)) {
+	            acceptChild(visitor, expr);
+	        }
+	        visitor.endVisit(this);
+			
+		}
+    }
 }

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/parser/SQLServerStatementParser.java

@@ -15,9 +15,12 @@
  */
 package com.alibaba.druid.sql.dialect.sqlserver.parser;
 
+import java.util.Collection;
 import java.util.List;
 
+import com.alibaba.druid.sql.ast.SQLExpr;
 import com.alibaba.druid.sql.ast.SQLName;
+import com.alibaba.druid.sql.ast.SQLObject;
 import com.alibaba.druid.sql.ast.SQLStatement;
 import com.alibaba.druid.sql.ast.expr.SQLIdentifierExpr;
 import com.alibaba.druid.sql.ast.expr.SQLQueryExpr;
@@ -43,6 +46,7 @@ import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerSetStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerSetTransactionIsolationLevelStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerUpdateStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerWaitForStatement;
+import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement.SQLServerParameter;
 import com.alibaba.druid.sql.parser.Lexer;
 import com.alibaba.druid.sql.parser.ParserException;
 import com.alibaba.druid.sql.parser.SQLSelectParser;
@@ -76,7 +80,7 @@ public class SQLServerStatementParser extends SQLStatementParser {
             SQLServerExecStatement execStmt = new SQLServerExecStatement();
             if (lexer.token() == Token.LPAREN) {
                 lexer.nextToken();
-                this.exprParser.exprList(execStmt.getParameters(), execStmt);
+                this.parseExecParameter(execStmt.getParameters(), execStmt);
                 accept(Token.RPAREN);
             } else {
                 SQLName sqlNameName = this.exprParser.name();
@@ -88,8 +92,8 @@ public class SQLServerStatementParser extends SQLStatementParser {
                 } else {
                     execStmt.setModuleName(sqlNameName);
                 }
-
-                this.exprParser.exprList(execStmt.getParameters(), execStmt);
+                
+                this.parseExecParameter(execStmt.getParameters(), execStmt);
             }
             statementList.add(execStmt);
             return true;
@@ -122,6 +126,44 @@ public class SQLServerStatementParser extends SQLStatementParser {
         
         return false;
     }
+    /**
+     * SQLServer parse Parameter statement support out type
+     * @author zz email:455910092@qq.com
+     * @date 2015-9-20
+     */
+    public void parseExecParameter(Collection<SQLServerParameter> exprCol, SQLObject parent)
+    {
+    	if (lexer.token() == Token.RPAREN || lexer.token() == Token.RBRACKET) {
+            return;
+        }
+
+        if (lexer.token() == Token.EOF) {
+            return;
+        }
+    	SQLServerParameter param=new SQLServerParameter();
+        SQLExpr expr=this.exprParser.expr();
+        expr.setParent(parent);
+        param.setExpr(expr);
+        if(lexer.token()==Token.OUT)
+    	{
+        	param.setType(true);
+    		accept(Token.OUT);
+    	}
+        exprCol.add(param);
+        while (lexer.token() == Token.COMMA) {
+        	lexer.nextToken();
+        	param=new SQLServerParameter();
+        	expr=this.exprParser.expr();
+            expr.setParent(parent);
+            param.setExpr(expr);
+        	if(lexer.token()==Token.OUT)
+        	{
+        		param.setType(true);
+        		accept(Token.OUT);
+        	}
+        	exprCol.add(param);
+        }
+    }
     
     public SQLStatement parseDeclare() {
         this.accept(Token.DECLARE);

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/visitor/SQLServerASTVisitor.java

@@ -26,6 +26,7 @@ import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerBlockStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerCommitStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerDeclareStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement;
+import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement.SQLServerParameter;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement.Else;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerInsertStatement;
@@ -117,5 +118,9 @@ public interface SQLServerASTVisitor extends SQLASTVisitor {
     boolean visit(SQLServerWaitForStatement x);
 
     void endVisit(SQLServerWaitForStatement x);
+    
+    boolean visit(SQLServerParameter x);
+
+    void endVisit(SQLServerParameter x);
 
 }

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/visitor/SQLServerASTVisitorAdapter.java

@@ -26,6 +26,7 @@ import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerBlockStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerCommitStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerDeclareStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement;
+import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement.SQLServerParameter;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement.Else;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerInsertStatement;
@@ -238,4 +239,16 @@ public class SQLServerASTVisitorAdapter extends SQLASTVisitorAdapter implements
         
     }
 
+	@Override
+	public boolean visit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		return false;
+	}
+
+	@Override
+	public void endVisit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		
+	}
+
 }

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/visitor/SQLServerOutputVisitor.java

@@ -35,6 +35,7 @@ import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerBlockStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerCommitStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerDeclareStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement;
+import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement.SQLServerParameter;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement.Else;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerInsertStatement;
@@ -690,4 +691,21 @@ public class SQLServerOutputVisitor extends SQLASTOutputVisitor implements SQLSe
     public void endVisit(SQLServerWaitForStatement x) {
         
     }
+
+	@Override
+	public boolean visit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		x.getExpr().accept(this);
+		if(x.getType())
+		{
+			print(" OUT");
+		}
+		return false;
+	}
+
+	@Override
+	public void endVisit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		
+	}
 }

src/main/java/com/alibaba/druid/sql/dialect/sqlserver/visitor/SQLServerSchemaStatVisitor.java

@@ -33,6 +33,7 @@ import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerBlockStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerCommitStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerDeclareStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement;
+import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerExecStatement.SQLServerParameter;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerIfStatement.Else;
 import com.alibaba.druid.sql.dialect.sqlserver.ast.stmt.SQLServerInsertStatement;
@@ -274,4 +275,16 @@ public class SQLServerSchemaStatVisitor extends SchemaStatVisitor implements SQL
 
     }
 
+	@Override
+	public boolean visit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		return false;
+	}
+
+	@Override
+	public void endVisit(SQLServerParameter x) {
+		// TODO Auto-generated method stub
+		
+	}
+
 }

src/main/java/com/alibaba/druid/sql/parser/Keywords.java

@@ -135,7 +135,7 @@ public class Keywords {
         map.put("GRANT", Token.GRANT);
         map.put("REVOKE", Token.REVOKE);
         
-        // MySql procedure by:zz
+        // MySql procedure: add by zz
         map.put("WHILE", Token.WHILE);
         map.put("DO", Token.DO);
         map.put("DECLARE", Token.DECLARE);
@@ -148,6 +148,8 @@ public class Keywords {
         map.put("CLOSE", Token.CLOSE);
         map.put("CURSOR", Token.CURSOR);
         map.put("FETCH", Token.FETCH);
+        map.put("OUT", Token.OUT);
+        map.put("INOUT", Token.INOUT);
         
         DEFAULT_KEYWORDS = new Keywords(map);
     }

src/main/java/com/alibaba/druid/sql/parser/Token.java

@@ -128,7 +128,7 @@ public enum Token {
     SHOW("SHOW"),
     REPLACE("REPLACE"),
     
-    // MySql procedure by:zz
+    // MySql procedure add by zz
     WHILE("WHILE"),
     DO("DO"),
     LEAVE("LEAVE"),
@@ -137,6 +137,8 @@ public enum Token {
     UNTIL("UNTIL"),
     OPEN("OPEN"),
     CLOSE("CLOSE"),
+    OUT("OUT"),
+    INOUT("INOUT"),
     
     //postgresql
     WINDOW("WINDOW"),

src/test/java/com/alibaba/druid/bvt/sql/mysql/MySqlCreateProcedureTest6.java

@@ -0,0 +1,155 @@
+/*
+ * Copyright 1999-2101 Alibaba Group Holding Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.alibaba.druid.bvt.sql.mysql;
+
+import java.util.List;
+
+import org.junit.Assert;
+
+import com.alibaba.druid.sql.MysqlTest;
+import com.alibaba.druid.sql.ast.SQLStatement;
+import com.alibaba.druid.sql.dialect.mysql.parser.MySqlStatementParser;
+import com.alibaba.druid.sql.dialect.mysql.visitor.MySqlSchemaStatVisitor;
+import com.alibaba.druid.stat.TableStat;
+import com.alibaba.druid.stat.TableStat.Column;
+
+/**
+ * 
+ * @Description: create procedure parameter type support
+ * @author zz email:455910092@qq.com
+ * @date 2015-9-14
+ * @version V1.0
+ */
+public class MySqlCreateProcedureTest6 extends MysqlTest {
+
+    public void test_0() throws Exception {
+    	String sql="create or replace procedure sp_name(level int,age int)"+
+				" begin"+
+    			" insert into test values(id,age);"+
+				" end";
+	
+    	MySqlStatementParser parser=new MySqlStatementParser(sql);
+    	List<SQLStatement> statementList = parser.parseStatementList();
+    	SQLStatement statemen = statementList.get(0);
+    	print(statementList);
+        Assert.assertEquals(1, statementList.size());
+
+        MySqlSchemaStatVisitor visitor = new MySqlSchemaStatVisitor();
+        statemen.accept(visitor);
+
+        System.out.println("Tables : " + visitor.getTables());
+        System.out.println("fields : " + visitor.getColumns());
+        System.out.println("coditions : " + visitor.getConditions());
+        System.out.println("orderBy : " + visitor.getOrderByColumns());
+        
+        Assert.assertEquals(1, visitor.getTables().size());
+        Assert.assertEquals(2, visitor.getColumns().size());
+        Assert.assertEquals(0, visitor.getConditions().size());
+
+        Assert.assertTrue(visitor.getTables().containsKey(new TableStat.Name("test")));
+        
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "id")));
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "age")));
+    }
+    public void test_1() throws Exception {
+    	String sql="create or replace procedure sp_name(in level int,in age int)"+
+				" begin"+
+    			" insert into test values(id,age);"+
+				" end";
+	
+    	MySqlStatementParser parser=new MySqlStatementParser(sql);
+    	List<SQLStatement> statementList = parser.parseStatementList();
+    	SQLStatement statemen = statementList.get(0);
+    	print(statementList);
+        Assert.assertEquals(1, statementList.size());
+
+        MySqlSchemaStatVisitor visitor = new MySqlSchemaStatVisitor();
+        statemen.accept(visitor);
+
+        System.out.println("Tables : " + visitor.getTables());
+        System.out.println("fields : " + visitor.getColumns());
+        System.out.println("coditions : " + visitor.getConditions());
+        System.out.println("orderBy : " + visitor.getOrderByColumns());
+        
+        Assert.assertEquals(1, visitor.getTables().size());
+        Assert.assertEquals(2, visitor.getColumns().size());
+        Assert.assertEquals(0, visitor.getConditions().size());
+
+        Assert.assertTrue(visitor.getTables().containsKey(new TableStat.Name("test")));
+        
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "id")));
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "age")));
+    }
+    public void test_2() throws Exception {
+    	String sql="create or replace procedure sp_name(out level int,out age int)"+
+				" begin"+
+    			" insert into test values(id,age);"+
+				" end";
+	
+    	MySqlStatementParser parser=new MySqlStatementParser(sql);
+    	List<SQLStatement> statementList = parser.parseStatementList();
+    	SQLStatement statemen = statementList.get(0);
+    	print(statementList);
+        Assert.assertEquals(1, statementList.size());
+
+        MySqlSchemaStatVisitor visitor = new MySqlSchemaStatVisitor();
+        statemen.accept(visitor);
+
+        System.out.println("Tables : " + visitor.getTables());
+        System.out.println("fields : " + visitor.getColumns());
+        System.out.println("coditions : " + visitor.getConditions());
+        System.out.println("orderBy : " + visitor.getOrderByColumns());
+        
+        Assert.assertEquals(1, visitor.getTables().size());
+        Assert.assertEquals(2, visitor.getColumns().size());
+        Assert.assertEquals(0, visitor.getConditions().size());
+
+        Assert.assertTrue(visitor.getTables().containsKey(new TableStat.Name("test")));
+        
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "id")));
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "age")));
+    }
+    public void test_3() throws Exception {
+    	String sql="create or replace procedure sp_name(inout level int,inout age int)"+
+				" begin"+
+    			" insert into test values(id,age);"+
+				" end";
+	
+    	MySqlStatementParser parser=new MySqlStatementParser(sql);
+    	List<SQLStatement> statementList = parser.parseStatementList();
+    	SQLStatement statemen = statementList.get(0);
+    	print(statementList);
+        Assert.assertEquals(1, statementList.size());
+
+        MySqlSchemaStatVisitor visitor = new MySqlSchemaStatVisitor();
+        statemen.accept(visitor);
+
+        System.out.println("Tables : " + visitor.getTables());
+        System.out.println("fields : " + visitor.getColumns());
+        System.out.println("coditions : " + visitor.getConditions());
+        System.out.println("orderBy : " + visitor.getOrderByColumns());
+        
+        Assert.assertEquals(1, visitor.getTables().size());
+        Assert.assertEquals(2, visitor.getColumns().size());
+        Assert.assertEquals(0, visitor.getConditions().size());
+
+        Assert.assertTrue(visitor.getTables().containsKey(new TableStat.Name("test")));
+        
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "id")));
+        Assert.assertTrue(visitor.getColumns().contains(new Column("test", "age")));
+    }
+    
+}

src/test/java/com/alibaba/druid/bvt/sql/sqlserver/SQLServerExec_Test.java

@@ -71,4 +71,28 @@ public class SQLServerExec_Test extends TestCase {
         Assert.assertEquals(0, visitor.getColumns().size());
     }
 
+    /**
+     * add by zz to test execute sp with out type
+     * @throws Exception
+     */
+    public void test_3() throws Exception {
+        String sql = "EXEC sp_rename @P0 OUT";
+        SQLServerStatementParser parser = new SQLServerStatementParser(sql);
+        SQLStatement stmt = parser.parseStatementList().get(0);
+        parser.match(Token.EOF);
+
+        SQLServerSchemaStatVisitor visitor = new SQLServerSchemaStatVisitor();
+        stmt.accept(visitor);
+
+        System.out.println("Tables : " + visitor.getTables());
+        System.out.println("fields : " + visitor.getColumns());
+        System.out.println("coditions : " + visitor.getConditions());
+        System.out.println("orderBy : " + visitor.getOrderByColumns());
+
+        String output = SQLUtils.toSQLString(stmt, JdbcConstants.SQL_SERVER);
+        Assert.assertEquals("EXEC sp_rename @P0 OUT", output);
+        
+        Assert.assertEquals(0, visitor.getTables().size());
+        Assert.assertEquals(0, visitor.getColumns().size());
+    }
 }