arpadboda 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_r249233347
########## 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: This exists in both http and raw, with different implementations, so I guess it's fine here as pure virtual. ---------------------------------------------------------------- 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