<div dir="ltr">On Sat, Dec 24, 2016 at 11:47 AM, Augie Fackler <span dir="ltr"><<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Dec 24, 2016, at 1:34 PM, Gregory Szorc <<a href="mailto:gregory.szorc@gmail.com" target="_blank">gregory.szorc@gmail.com</a>> wrote:</div><br class="m_6254790256775393516Apple-interchange-newline"><div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">Where do we stand with this series? I was waiting for feedback, but none arrived.<br></div></div></blockquote><div><br></div></span><div>I believe I had asked that others look at it, and hearing nothing, we should conclude nobody else cares and move on and queue it with whatever little bits I had to say on this v2. Does that work for you?</div><div><br></div><div>(I did give you feedback, right?)</div><span class=""><br><blockquote type="cite"><div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">I'd like to get zstd into the wire protocol and revlogs for 4.1, otherwise it isn't very useful (I've been holding off patchbombing the revlog series because I already have enough things in flight). With the holidays, we effectively have 2 weeks before freeze.</div></div></blockquote><div><br></div></span><div>Agreed.</div><span class=""><br><blockquote type="cite"><div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><br>FWIW, getting reviews this cycle has been extremely frustrating to me. I was planning on landing zstd, `hg debugupgraderepo`, `hg display`, and various polish work for more robust zstd integration in the 4.1 release. It's now looking like a good chunk of that work won't make it. The most frustrating part is that most of the code was ready in November :/ I think there needs to be a discussion about scaling code review (no later than the next sprint) because it's been obvious for a few months now that things aren't as well-oiled as they were several months ago.</div></div></blockquote></span></div><br><div>Also agreed. Note that this one got lost while you were on vacation (or something?), and debugupgraderepo I left for marmoute at his explicit request, but I haven’t seen movement there.</div></div></blockquote></div><br></div><div class="gmail_extra">Yeah, I did take a bit of a pause on this series because a) I was waiting for feedback b) I had a week long business trip c) I took a mini vacation the week after.<br><br></div><div class="gmail_extra">But since silence is consent on this series, I'll resume steaming ahead. Thanks for the clarification.<br></div></div>