<html><head></head><body><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div id="ydp4f2d9a80yiv5911623007"><div><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div style="font-family:verdana, helvetica, sans-serif;font-size:16px;"><div></div>
<div>Hi Alex,</div><div><br clear="none"></div><div>I agree adding additional functionality to merge smaller range objects into larger ones is a better approach. However, it requires an additional module and file structure to bookkeep cached ranges of each URL and a way to rebuild the bookkeeping index after a restart. I am not sure if the additional effort will worth it since squid already supports partial content from cache if the full file is loaded in cache.</div><div><br clear="none"></div><div>Thanks for pointing out that some use cases may not work with my implementation. Now I know my solution is good for a limited use case (repeating requests with similar ranges) but not at a global scale. In that case, I think it will be better not to merge it into the main branch.</div><div><br clear="none"></div><div><span></span><h3 class="ydp4f2d9a80yiv5911623007ydp9980a4ear" style="font-size:18px;font-weight:normal;margin:0px;padding:0px;white-space:nowrap;color:rgb(34, 34, 34);font-family:arial, sans-serif;"><a shape="rect" href="https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=2&cad=rja&uact=8&ved=2ahUKEwjB0_rA4rndAhULT98KHXwwBoQQFjABegQICRAB&url=https%3A%2F%2Fnews.ycombinator.com%2Fitem%3Fid%3D1659912&usg=AOvVaw0Nr4fFgdmeHYahFt2TdKwH" style="color:rgb(102, 0, 153);cursor:pointer;" rel="nofollow" target="_blank"></a></h3><span></span><h3 class="ydp4f2d9a80yiv5911623007ydpce95c16br" style="font-size:18px;font-weight:normal;margin:0px;padding:0px;white-space:nowrap;color:rgb(34, 34, 34);font-family:arial, sans-serif;"><a shape="rect" href="https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=2&cad=rja&uact=8&ved=2ahUKEwjB0_rA4rndAhULT98KHXwwBoQQFjABegQICRAB&url=https%3A%2F%2Fnews.ycombinator.com%2Fitem%3Fid%3D1659912&usg=AOvVaw0Nr4fFgdmeHYahFt2TdKwH" style="color:rgb(102, 0, 153);cursor:pointer;" rel="nofollow" target="_blank"></a></h3>Thank your very much for your detailed explanation.</div></div></div></div></div></div></div></div></div></div></div><div class="ydped067502yiv5911623007yqt1971680399" id="ydped067502yiv5911623007yqtfd20593"><div><br clear="none"></div><div>Benson</div><div></div></div><div class="ydped067502yiv5911623007yqt1971680399" id="ydped067502yiv5911623007yqtfd62214"><div class="ydped067502yiv5911623007ydp9a749293yahoo_quoted" id="ydped067502yiv5911623007ydp9a749293yahoo_quoted_7607457129"><div style="font-family:'Helvetica Neue', Helvetica, Arial, sans-serif;font-size:13px;color:#26282a;"><div><div dir="ltr"><div class="ydped067502yiv5911623007ydp9a749293yqt3716328950" id="ydped067502yiv5911623007ydp9a749293yqtfd16914"><br clear="none"></div></div></div>
</div>
</div></div></div></body></html>