Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-04-04 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/#review170973 --- Ship it! Ship It! - Josh Elser On April 3, 2017, 8:12 p.m.,

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-04-03 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- (Updated April 3, 2017, 8:12 p.m.) Review request for hive and Josh Elser. Ch

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-04-03 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- (Updated April 3, 2017, 8:08 p.m.) Review request for hive and Josh Elser. Ch

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-31 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- (Updated March 31, 2017, 11:19 p.m.) Review request for hive and Josh Elser.

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-29 Thread Mike Fagan
> On Feb. 6, 2017, 6:08 p.m., Josh Elser wrote: > > accumulo-handler/src/java/org/apache/hadoop/hive/accumulo/AccumuloDefaultIndexScanner.java > > Lines 43 (patched) > > > > > > A few questions: > > > > * Wha

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-29 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- (Updated March 30, 2017, 5:29 a.m.) Review request for hive and Josh Elser. C

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-29 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/#review170427 --- accumulo-handler/src/java/org/apache/hadoop/hive/accumulo/Accumul

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-29 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/#review170404 --- This is looking very nice, Mike! I'm going to pull down what you h

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-03-06 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- (Updated March 6, 2017, 4:24 p.m.) Review request for hive and Josh Elser. Ch

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-02-07 Thread Peter Vary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/#review164491 --- Hi, Thanks for the patch. I am not an expert on Accumulo, so I le

Re: Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-02-06 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/#review164360 --- accumulo-handler/src/java/org/apache/hadoop/hive/accumulo/Accumul

Review Request 56290: HIVE-15795: Add Accumulo Index Table Support

2017-02-06 Thread Mike Fagan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56290/ --- Review request for hive. Repository: hive-git Description --- HIVE-15795