-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18843/
-----------------------------------------------------------

Review request for Tajo.


Repository: tajo


Description
-------

See the title. SQLParser cannot parse the INET4 type.
I fixed the problem of parser and some missing codes for processing of INET4 
type.
I also added two functions of country() and in_country() for INET4 type.
Lastly, I added the TestNetTypes class to verify the query processing of INET4 
type.


Diffs
-----

  
tajo-core/tajo-core-backend/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
 db04d4b 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/Country.java
 9e28b55 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/CountryInet4.java
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/CountryText.java
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/InCountry.java
 1cac624 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/InCountryInet4.java
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/function/InCountryText.java
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
 29e4d43 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/ExprsVerifier.java
 358cabd 
  
tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/planner/RangePartitionAlgorithm.java
 5bff857 
  
tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/query/TestNetTypes.java
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/dataset/TestNetTypes/table1.tbl 
PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/dataset/TestNetTypes/table2.tbl 
PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/table1_ddl.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/table2_ddl.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/testGroupby.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/testGroupby2.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/testJoin.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/testSelect.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/queries/TestNetTypes/testSort.sql
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/results/TestNetTypes/testGroupby.result
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/results/TestNetTypes/testGroupby2.result
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/results/TestNetTypes/testJoin.result
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/results/TestNetTypes/testSelect.result
 PRE-CREATION 
  
tajo-core/tajo-core-backend/src/test/resources/results/TestNetTypes/testSort.result
 PRE-CREATION 

Diff: https://reviews.apache.org/r/18843/diff/


Testing
-------

mvn verify


Thanks,

Jihoon Son

Reply via email to