This is an automated email from the ASF dual-hosted git repository.

nicknezis pushed a commit to branch nicknezis/cppcheck-update
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git

commit 43f0b9a73ff45414a1594998fd528077c9abdcaa
Author: Nicholas Nezis <[email protected]>
AuthorDate: Sat Mar 26 00:56:05 2022 -0400

    Fixed a bug that the new cppcheck helped find
---
 heron/stmgr/tests/cpp/server/stmgr_unittest.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/heron/stmgr/tests/cpp/server/stmgr_unittest.cpp 
b/heron/stmgr/tests/cpp/server/stmgr_unittest.cpp
index 535d56c..58f4e37 100644
--- a/heron/stmgr/tests/cpp/server/stmgr_unittest.cpp
+++ b/heron/stmgr/tests/cpp/server/stmgr_unittest.cpp
@@ -771,7 +771,7 @@ TEST(StMgr, test_pplan_decode) {
               std::find(common.stmgr_ports_.begin(), common.stmgr_ports_.end(),
                         pplan0->stmgrs(i).data_port()));
     EXPECT_NE(common.local_data_ports_.end(),
-              std::find(common.stmgr_ports_.begin(), common.stmgr_ports_.end(),
+              std::find(common.local_data_ports_.begin(), 
common.local_data_ports_.end(),
                         pplan0->stmgrs(i).local_data_port()));
   }
   EXPECT_EQ(pplan0->instances_size(), common.num_stmgrs_ * 
num_workers_per_stmgr_);

Reply via email to