mmartell commented on a change in pull request #780:
URL: https://github.com/apache/geode-native/pull/780#discussion_r607208728
##########
File path: cppcache/integration-test/ThinClientHelper.hpp
##########
@@ -134,7 +134,7 @@ const char* vals[] = {"Value-1", "Value-2", "Value-3",
const char* nvals[] = {"New Value-1", "New Value-2", "New Value-3",
"New Value-4", "New Value-5", "New Value-6"};
-const char* regionNames[] = {"DistRegionAck", "DistRegionNoAck"};
+const char* regionNames[] = {"DistRegionAck", "not-used"};
Review comment:
Lots of tests use this header. Changing to a scalar caused lots of
compile issues (mostly in disabled tests). Suggest we leave this file
untouched, as probably too much work to fix compile issues.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
