> On Feb. 13, 2014, 10:39 p.m., Hyunsik Choi wrote: > > tajo-core/tajo-core-backend/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4, > > line 1049 > > <https://reviews.apache.org/r/17642/diff/4/?file=484187#file484187line1049> > > > > The class name was renamed, but it still remains the old name. > > > > I would like to suggest you to refer SQL BNF at > > http://savage.net.au/SQL/sql-2003-2.bnf.html#qualified asterisk. > > > > I usually finds proper rune names from SQL BNF. I hope that it would be > > helpful to you.
Thanks for your recommendation. I fixed. > On Feb. 13, 2014, 10:39 p.m., Hyunsik Choi wrote: > > tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/LogicalPlan.java, > > line 317 > > <https://reviews.apache.org/r/17642/diff/4/?file=484193#file484193line317> > > > > It seems to be invoked from only > > LogicalPlanPreprocessor::resolveAsterisk(). If so, it would be nice to move > > it to LogicalPlanPreprocessor. I moved it. Thanks! - Jihoon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17642/#review34360 ----------------------------------------------------------- On Feb. 14, 2014, 1:03 a.m., Jihoon Son wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > 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 > >
