kaisun2000 commented on PR #15373: URL: https://github.com/apache/druid/pull/15373#issuecomment-1830831818
> Thanks @gianm and @kaisun2000, for the context. One last question: does it need to be configurable? I don't think so. We can either hardcode it to 30 seconds or we can reuse the same config option that already exists. > > ``` > druid.segmentCache.dropSegmentDelayMillis > ``` > > There is no need of a per ingestion configuration here. @abhishekagarwal87, I am pretty open to any suggestions. As you said, maybe a per Peon/middlemanager configuration is just as good. However, one peon takes one ingestion spec and that is why I put it in ingestion spec. Can you supply another example of per middle manager configuration which will be carried to peons? I can also change the configuration that way. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
