paul-rogers opened a new pull request #1930: DRILL-7487: Removes the unused OUT_OF_MEMORY iterator status URL: https://github.com/apache/drill/pull/1930 Drill has long supported the `OUT_OF_MEMORY` iterator status. The idea is that an operator can realize it has encountered memory pressure and ask its downstream operator to free up some memory. However, an inspection of the code shows that the status is actually sent in only one place (`UnorderedReceiverBatch`), and then only in response to the operator hitting its allocator limit (which no other batch can do anything about.) This PR removes the `OUT_OF_MEMORY` status and all uses. All operators tried to handle the status, but no operator (except the one above) actually sent it. Adds a stub implementation for an OOM strategy (borrowed from Presto) that might actually work. See JIRA ticket for full explanation. Tests: Ran all exec tests to verify the revised code builds and runs.
---------------------------------------------------------------- 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] With regards, Apache Git Services
