phrocker commented on a change in pull request #470: MINIFICPP-706 - RawSiteToSite: remove code duplication URL: https://github.com/apache/nifi-minifi-cpp/pull/470#discussion_r249102304
########## File path: libminifi/include/sitetosite/SiteToSiteClient.h ########## @@ -221,12 +221,8 @@ class SiteToSiteClient : public core::Connectable { // deleteTransaction virtual void deleteTransaction(std::string transactionID); - virtual void tearDown(); - - // write Request Type - virtual int writeRequestType(RequestType type); - // read Request Type - virtual int readRequestType(RequestType &type); + virtual void tearDown() = 0; Review comment: i don't think this is needed at all. is it used elsewhere? Teardown is protocol specific of when and how it occurs. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services