stak...@cloudera.com has posted comments on this change.

Change subject: IMPALA-3980: qgen: re-enable Hive as a target database
......................................................................


Patch Set 4:

> > I'd like you to run the small suite of cluster tests, but you
 > can't
 > > yet until you rebase on top of https://gerrit.cloudera.org/#/c/4404/
 > > I realize there aren't that many, and they may not find anything,
 > > but it's a habit we need to start. Thanks.
 > 
 > Will do, looks like your patch just got merged. I will rebase my
 > patch and then run the tests. Will update this review once I'm
 > done.

@Michael, rebased this patch and ran all the units tests (test_cluster.py and 
test_query_objects.py). All tests passed.

-- 
To view, visit http://gerrit.cloudera.org:8080/4011
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb1199b50a5b65c21de7876fb70cc03bda1a9b46
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: stak...@cloudera.com
Gerrit-Reviewer: David Knupp <dkn...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Michael Brown <mi...@cloudera.com>
Gerrit-Reviewer: Taras Bobrovytsky <tbobrovyt...@cloudera.com>
Gerrit-Reviewer: stak...@cloudera.com
Gerrit-HasComments: No

Reply via email to