-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17642/
-----------------------------------------------------------
(Updated Feb. 14, 2014, 1:03 a.m.)
Review request for Tajo, Jung JaeHwa and Hyunsik Choi.
Changes
-------
Thanks guys!
I reflect Hyunsik's comments and add more tests for SQLAnalyzer and
HiveConveter.
Also, I fixed a bug reported in TAJO-597.
Now, Tajo can process the following query properly.
> select length(r_comment), *, c_custkey*10 from customer, region;
Bugs: TAJO-554
https://issues.apache.org/jira/browse/TAJO-554
Repository: tajo
Description
-------
See the title. LogicalPlanner should allow the following case.
SELECT *, col1 + 10, col2 FROM ...
In this patch, the asterisk expression is replaced with the target relation's
columns in LogicalPlanPreprocessor.
Diffs (updated)
-----
tajo-algebra/src/main/java/org/apache/tajo/algebra/OpType.java 84f784e
tajo-algebra/src/main/java/org/apache/tajo/algebra/Projection.java 3d9f8a6
tajo-algebra/src/main/java/org/apache/tajo/algebra/QualifiedAsteriskExpr.java
PRE-CREATION
tajo-core/tajo-core-backend/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
ab6bff6
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/HiveConverter.java
a8a555b
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
493f892
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/AlgebraVisitor.java
1c710dc
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/BaseAlgebraVisitor.java
25ee316
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanPreprocessor.java
099e462
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlanner.java
d929218
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/parser/TestHiveConverter.java
e090c89
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java
8d7cb94
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/planner/TestLogicalPlanner.java
81f57d4
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestJoinQuery.java
dc951cf
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestSelectQuery.java
4d852fc
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestSortQuery.java
b6e439a
tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testCrossJoinWithAsterisk1.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testCrossJoinWithAsterisk2.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testCrossJoinWithAsterisk3.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/TestJoinQuery/testCrossJoinWithAsterisk4.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/TestSelectQuery/testSelectAsterisk4.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/TestSortQuery/testAsterisk.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/default/asterisk_1.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/default/asterisk_2.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/default/asterisk_3.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/queries/default/asterisk_4.sql
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestJoinQuery/testCrossJoinWithAsterisk1.result
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestJoinQuery/testCrossJoinWithAsterisk2.result
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestJoinQuery/testCrossJoinWithAsterisk3.result
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestJoinQuery/testCrossJoinWithAsterisk4.result
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestSelectQuery/testSelectAsterisk4.result
PRE-CREATION
tajo-core/tajo-core-backend/src/test/resources/results/TestSortQuery/testAsterisk.result
PRE-CREATION
Diff: https://reviews.apache.org/r/17642/diff/
Testing
-------
mvn verify
Thanks,
Jihoon Son
