Review Request: HIVE-1803: Implement bitmap indexing in Hive (new review starting from patch 8)

2011-03-28 Thread John Sichi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/530/ --- Review request for hive. Summary --- New review request for HIVE-1803.8

Re: Review Request: HIVE-1803: Implement bitmap indexing in Hive (new review starting from patch 6)

2011-03-22 Thread Marquis Wang
On None, John Sichi wrote: ql/src/java/org/apache/hadoop/hive/ql/index/compact/HiveCompactIndexInputFormat.java, line 45 https://reviews.apache.org/r/481/diff/1/?file=13771#file13771line45 I'm confused about how the backwards compatibility works for the index filename

Review Request: HIVE-1803: Implement bitmap indexing in Hive (new review starting from patch 6)

2011-03-08 Thread John Sichi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/481/ --- Review request for hive. Summary --- Review board was giving me grief

Re: Review Request: HIVE-1803: Implement bitmap indexing in Hive

2011-03-07 Thread Marquis Wang
On None, John Sichi wrote: ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java, line 50 https://reviews.apache.org/r/466/diff/1/?file=13567#file13567line50 The point of factoring out the compact index handler base class was so that you could eliminate

Re: Review Request: HIVE-1803: Implement bitmap indexing in Hive

2011-03-05 Thread M IS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/466/#review304 ---

Review Request: HIVE-1803: Implement bitmap indexing in Hive

2011-03-04 Thread John Sichi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/466/ --- Review request for hive. Summary --- Review by JVS. This addresses bug