Thanks a lot to Ashutosh for reviewing and committing HIVE-3384 to trunk. We still have the following patches outstanding that would help the Shark project a lot -- your feedback / reviewing / committing would be much appreciated!
- https://reviews.facebook.net/D6879 (HIVE-3731<https://issues.apache.org/jira/browse/HIVE-3731>): adding an Ant target to create a Debian package, which allows deploying the patched version of Hive alongside Shark on Debian systems. - https://reviews.facebook.net/D7155 (HIVE-3772<https://issues.apache.org/jira/browse/HIVE-3772>): a one-line concurrency bug fix in LazyBinaryUtils (patch by Reynold Xin). Thanks, Mikhail On Tue, Nov 27, 2012 at 7:08 PM, Ashutosh Chauhan <hashut...@apache.org>wrote: > Hi Mikhail, > > I will take a look into those jiras. > > Thanks, > Ashutosh > > On Tue, Nov 27, 2012 at 11:43 AM, Mikhail Bautin < > bautin.mailing.li...@gmail.com> wrote: > > > Hello, > > > > There are the following review requests pending that are very useful for > > the Shark project (http://shark.cs.berkeley.edu/). It would be great if > > someone could take a look and help us get these JIRAs committed. > > > > - https://reviews.facebook.net/D6879 > > (HIVE-3731<https://issues.apache.org/jira/browse/HIVE-3731>): > > adding an Ant target to create a Debian package, which allows > deploying > > the > > patched version of Hive alongside Shark on Debian systems. > > - https://reviews.facebook.net/D7005 > > (HIVE-3748<https://issues.apache.org/jira/browse/HIVE-3748>): > > making QTestUtil work correctly when running the test suite, which > helps > > with running Hive/Shark unit tests from using Maven. > > > > In addition, the following JIRA would make a lot easier to work with Hive > > for anyone who is using JDK 1.7: > > > > - https://reviews.facebook.net/D6873 > > (HIVE-3384<https://issues.apache.org/jira/browse/HIVE-3384>): > > HIVE JDBC module won't compile under JDK1.7 as new methods added in > JDBC > > specification > > > > Your help in reviewing/committing these patches is greatly appreciated! > > > > Thanks, > > Mikhail > > >
