[GEODE-5993] QueryMonitor can throw RejectedExecutionException due to race

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.8.0
• Component/s: querying
• Labels: pull-request-available
• Assignee: Ryan McMahon
• Reporter: Ryan McMahon
• Created: 06/Nov/18 00:46
• Updated: 13/Dec/18 03:45
• Resolved: 13/Nov/18 17:12

Description

It is possible for a RejectedExecutionException to be thrown due to the following race:

Thread 1)

• Call monitorQueryThread() and pass LOW_MEMORY check

Thread 2)

• Call cancelAllQueriesDueToMemory()
• Shutdown executor

Thread 1)

• Schedule cancellation task results in RejectedExecutionException

Issue Links

• links to: GitHub Pull Request #2818