[FEATURE][ML] CDataFrameAnalyzer.handleRecord should return true on success

dimitris-athanasiou · dimitris-athanasiou · commit 295f9c3a6f8f · 2018-12-12T14:42:33.000Z
diff --git a/lib/api/CDataFrameAnalyzer.cc b/lib/api/CDataFrameAnalyzer.cc
@@ -87,7 +87,7 @@ bool CDataFrameAnalyzer::handleRecord(const TStrVec& fieldNames, const TStrVec&
     }
 
     this->addRowToDataFrame(fieldValues);
-    return false;
+    return true;
 }
 
 void CDataFrameAnalyzer::receivedAllRows() {

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ bool CDataFrameAnalyzer::handleRecord(const TStrVec& fieldNames, const TStrVec&`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`this->addRowToDataFrame(fieldValues);`
`90`		`- return false;`
	`90`	`+ return true;`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`void CDataFrameAnalyzer::receivedAllRows() {`