You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "kgyrtkirk (via GitHub)" <gi...@apache.org> on 2025/04/15 19:16:47 UTC
[PR] [Backport] Fix two issues where CursorHolders were not closed on failure of asCursor(). (#17915) (druid)
kgyrtkirk opened a new pull request, #17925:
URL: https://github.com/apache/druid/pull/17925
* Fix two issues where CursorHolders were not closed on failure of asCursor().
One in TopNQueryEngine (native) and one in ScanQueryFrameProcessor (MSQ).
This is especially important because asCursor() itself can fail in ways that leave the CursorHolder with resources that must be closed.
* Fix style.
(cherry picked from commit 0e81a19add0ccf4ed441d7c6549164b03cb901b0)
--
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
Re: [PR] [Backport] Fix two issues where CursorHolders were not closed on failure of asCursor(). (#17915) (druid)
Posted by "kgyrtkirk (via GitHub)" <gi...@apache.org>.
kgyrtkirk merged PR #17925:
URL: https://github.com/apache/druid/pull/17925
--
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