You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Prasad Mujumdar <pr...@cloudera.com> on 2013/12/12 08:44:01 UTC

Review Request 16207: HIVE-1466: Add NULL DEFINED AS to ROW FORMAT specification

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

Review request for hive.


Bugs: HIVE-1466
    https://issues.apache.org/jira/browse/HIVE-1466


Repository: hive-git


Description
-------

Support configurable null format for tables and writing out to directory.
Using a non-default null format is a bit while creating a table, and pretty much impossible to export the data to local filesystem using insert overwrite directory.
The patch enhances the SQL syntax to support 'NULL DEFINED AS' construct for create table as well as insert overwrite directory.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java d32be59 
  ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 4b7fc73 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveLexer.g 366b714 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 5e5b8cf 
  ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 8cf5ad6 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java d0a0ec7 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 93b4181 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java b6097b1 
  ql/src/test/queries/clientpositive/nullformat.q PRE-CREATION 
  ql/src/test/queries/clientpositive/nullformatdir.q PRE-CREATION 
  ql/src/test/results/clientpositive/nullformat.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/nullformatdir.q.out PRE-CREATION 

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


Testing
-------

Added new tests.


Thanks,

Prasad Mujumdar