Re: Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-17 Thread Xuefu Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/ --- (Updated March 18, 2014, 3:53 a.m.) Review request for hive. Bugs: HIVE-6677

Re: Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-17 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/#review37516 --- Looks fine to me. A minor formatting nits, feel free to ignore. hb

Re: Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-17 Thread Xuefu Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/#review37502 --- hbase-handler/src/test/org/apache/hadoop/hive/hbase/TestLazyHBaseOb

Re: Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-17 Thread Swarnim Kulkarni
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/#review37500 --- hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseSerDeParam

Re: Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-16 Thread Xuefu Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/ --- (Updated March 16, 2014, 5:33 p.m.) Review request for hive. Bugs: HIVE-6677

Review Request 19265: HIVE-6677: HBaseSerDe needs to be refactored

2014-03-16 Thread Xuefu Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19265/ --- Review request for hive. Bugs: HIVE-6677 https://issues.apache.org/jira/bro