You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@heron.apache.org by ni...@apache.org on 2022/03/26 04:56:38 UTC
[incubator-heron] 03/03: Fixed a bug that the new cppcheck helped find
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 <ni...@gmail.com>
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_);