----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25238/ -----------------------------------------------------------
(Updated Oct. 15, 2014, 12:45 a.m.) Review request for hive. Changes ------- Removed String mapping Bugs: HIVE-2828 https://issues.apache.org/jira/browse/HIVE-2828 Repository: hive-git Description ------- Originated from HIVE-2781 and not accepted, but I think this could be helpful to someone. By using special column notation ':timestamp' in HBASE_COLUMNS_MAPPING, user might access timestamp value in hbase KeyValue. {code} CREATE TABLE hbase_table (key int, value string, time timestamp) STORED BY 'org.apache.hadoop.hive.hbase.HBaseStorageHandler' WITH SERDEPROPERTIES ("hbase.columns.mapping" = ":key,cf:string,:timestamp") {code} Diffs (updated) ----- hbase-handler/src/java/org/apache/hadoop/hive/hbase/ColumnMappings.java fbd1308 hbase-handler/src/java/org/apache/hadoop/hive/hbase/DefaultHBaseKeyFactory.java 12c5377 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseRowSerializer.java c6c42b4 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseSerDe.java aedd843 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseSerDeParameters.java 9efa494 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java feb3cd1 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HiveHBaseInputFormatUtil.java 5aa1d796 hbase-handler/src/java/org/apache/hadoop/hive/hbase/HiveHBaseTableInputFormat.java 4ac0803 hbase-handler/src/java/org/apache/hadoop/hive/hbase/LazyHBaseRow.java 6ac8423 hbase-handler/src/test/org/apache/hadoop/hive/hbase/TestLazyHBaseObject.java 9a31f0f hbase-handler/src/test/queries/positive/hbase_timestamp.q PRE-CREATION hbase-handler/src/test/results/positive/hbase_timestamp.q.out PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/index/IndexPredicateAnalyzer.java 960fc1d serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java 0ab27ff Diff: https://reviews.apache.org/r/25238/diff/ Testing ------- Thanks, Navis Ryu