Thomas Tauber-Marshall has posted comments on this change. Change subject: Initial structure for the C++ hiveserver2 client. ......................................................................
Patch Set 16: (3 comments) http://gerrit.cloudera.org:8080/#/c/2645/16/src/hs2client/thrift-internal.cc File src/hs2client/thrift-internal.cc: Line 178: at > at() may throw an exception, which we don't do in our C++. Probably just as Done http://gerrit.cloudera.org:8080/#/c/2645/16/src/hs2client/thrift-internal.h File src/hs2client/thrift-internal.h: Line 58: // Converts a TTypeDesc to a ColumnType. Currently only primitive types are supported. > please mention why this returns a heap allocated object (polymorphism). Done http://gerrit.cloudera.org:8080/#/c/2645/16/src/hs2client/types.h File src/hs2client/types.h: Line 61: : virtual bool IsCharacterType() const { return false; } : virtual bool IsDecimalType() const { return false; } : virtual bool IsPrimitiveType() const { return false; } > I see why these are here, but I think we should avoid having to put things Done -- To view, visit http://gerrit.cloudera.org:8080/2645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7be82d5897ad6093116915b17924674c4ae508a0 Gerrit-PatchSet: 16 Gerrit-Project: hs2client Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Wes McKinney <[email protected]> Gerrit-HasComments: Yes
