[GEODE-5589] lastResultReceived is set to true well before the last message is sent

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.7.0
• Component/s: functions
• Labels: pull-request-available
• Assignee: Nabarun Nag
• Reporter: Nabarun Nag
• Created: 15/Aug/18 22:43
• Updated: 03/Oct/18 21:37
• Resolved: 17/Aug/18 16:32

Description

Issue:

• In the method lastResult we immediately set the lastResultReceived to be true.
• If an exception occurs after the flag is set and before the message is sent (like security exception)
• Then this exception will never be sent to the client because the lastResultReceived is set and clients will be stuck waiting for the function results.

Solution:

• set the flag after the message is sent.

Affected code :

• ServerToClientFunctionResultSender65.java
• ServerToClientFunctionResultSender.java

Issue Links

• links to: GitHub Pull Request #2341