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

Reply via email to