You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by "gianm (via GitHub)" <gi...@apache.org> on 2026/02/22 19:22:04 UTC
[PR] SegementCacheManager: acquireCachedSegment with SegmentId. (druid)
gianm opened a new pull request, #19044:
URL: https://github.com/apache/druid/pull/19044
Prior to this patch, acquireCachedSegment required the entire DataSegment object. This patch loosens the requirement to just the SegmentId.
This is useful under MSQ, where caching is fully on-demand (no load rules) and the full DataSegment object is not necessarily available in the local timeline, even if a segment is cached locally. See the change in RegularLoadableSegment.
--
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] SegementCacheManager: acquireCachedSegment with SegmentId. (druid)
Posted by "gianm (via GitHub)" <gi...@apache.org>.
gianm commented on PR #19044:
URL: https://github.com/apache/druid/pull/19044#issuecomment-3947477213
The failure was in Docker tests due to [19047](https://github.com/apache/druid/pull/19047), merging anyway.
--
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] SegementCacheManager: acquireCachedSegment with SegmentId. (druid)
Posted by "gianm (via GitHub)" <gi...@apache.org>.
gianm merged PR #19044:
URL: https://github.com/apache/druid/pull/19044
--
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