[
https://issues.apache.org/jira/browse/GEODE-2657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15932816#comment-15932816
]
ASF GitHub Bot commented on GEODE-2657:
---------------------------------------
Github user pivotal-jbarrett commented on a diff in the pull request:
https://github.com/apache/geode-native/pull/63#discussion_r106930126
--- Diff:
src/cppcache/integration-test/testThinClientPoolExecuteFunction.cpp ---
@@ -1506,48 +1353,10 @@ void runFunctionExecution(bool isEndpoint) {
CALL_TASK(StopC1);
CALL_TASK(CloseServers);
CALL_TASK(CloseLocator1);
-
--- End diff --
Like I said in the review request. This test was split out into tho two
tests. The test had two different tests crammed into one file. The big delete
in this file is all the crap that became the second test.
> Execute Region Function sends incorrect message format
> ------------------------------------------------------
>
> Key: GEODE-2657
> URL: https://issues.apache.org/jira/browse/GEODE-2657
> Project: Geode
> Issue Type: Bug
> Components: native client
> Reporter: David Kimura
> Assignee: Jacob S. Barrett
>
> `TcrMessageExecuteRegionFunction` is missing a call to set `m_hasResults`.
> This causes the message response to be parsed synchronously and not
> asynchronously chunked. The synchronous parser does not support this message
> type and it barfs.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
