Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11493 )
Change subject: Add thrift module for common thrift utilities ...................................................................... Patch Set 2: As a reminder, this thrift code doesn't contain any unit tests because it can only be exercised by way of connecting to an external server (since we don't have the necessary code to run a Thrift server ourselves). So, it's well exercised by the existing HMS tests. -- To view, visit http://gerrit.cloudera.org:8080/11493 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6f6f843f42b37cb1170df03da01fc0790fe94acb Gerrit-Change-Number: 11493 Gerrit-PatchSet: 2 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Fri, 21 Sep 2018 23:03:16 +0000 Gerrit-HasComments: No
