You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by gr...@apache.org on 2016/11/17 18:01:52 UTC
phoenix git commit: Fix extension of SqlParserTest
Repository: phoenix
Updated Branches:
refs/heads/calcite e003c579c -> 99b8a02b5
Fix extension of SqlParserTest
Override the appropriate method of SqlParserTest in
PhoenixSqlParserTest.
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/99b8a02b
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/99b8a02b
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/99b8a02b
Branch: refs/heads/calcite
Commit: 99b8a02b5ea15f6c14ba5a0a72059a9554862003
Parents: e003c57
Author: Gabriel Reid <ga...@ngdata.com>
Authored: Thu Nov 17 19:00:50 2016 +0100
Committer: Gabriel Reid <ga...@ngdata.com>
Committed: Thu Nov 17 19:00:50 2016 +0100
----------------------------------------------------------------------
.../org/apache/phoenix/calcite/PhoenixSqlParserTest.java | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/99b8a02b/phoenix-core/src/test/java/org/apache/phoenix/calcite/PhoenixSqlParserTest.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/test/java/org/apache/phoenix/calcite/PhoenixSqlParserTest.java b/phoenix-core/src/test/java/org/apache/phoenix/calcite/PhoenixSqlParserTest.java
index 45ebeb9..846a8ab 100644
--- a/phoenix-core/src/test/java/org/apache/phoenix/calcite/PhoenixSqlParserTest.java
+++ b/phoenix-core/src/test/java/org/apache/phoenix/calcite/PhoenixSqlParserTest.java
@@ -19,7 +19,7 @@ package org.apache.phoenix.calcite;
import java.io.IOException;
-import org.apache.calcite.sql.parser.SqlParser;
+import org.apache.calcite.sql.parser.SqlParserImplFactory;
import org.apache.calcite.sql.parser.SqlParserTest;
import org.apache.phoenix.calcite.parser.PhoenixParserImpl;
import org.junit.Test;
@@ -45,10 +45,8 @@ public class PhoenixSqlParserTest extends SqlParserTest {
}
@Override
- protected SqlParser getSqlParser(String sql) {
- return SqlParser.create(sql,
- SqlParser.configBuilder().setParserFactory(PhoenixParserImpl.FACTORY)
- .build());
+ protected SqlParserImplFactory parserImplFactory() {
+ return PhoenixParserImpl.FACTORY;
}
@Override