You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/10/25 05:52:49 UTC

[GitHub] [druid] gianm commented on pull request #13259: Fix two sources of SQL statement leaks.

gianm commented on PR #13259:
URL: https://github.com/apache/druid/pull/13259#issuecomment-1290022887

   I tested this patch by running `DruidAvaticaProtobufHandlerTest#testConcurrentQueries` in a loop. Prior to this fix, it would fail with OOME after 9–10 iterations. After this fix, it ran for 100 iterations before I canceled it. I checked a heap dump and did not see any statement objects beyond the ones that were currently executing.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org