larsrc-google

Looks good overall, I'll add some smaller comments soon.

Ugh, not again. There has been a number of failures on the Darwin tests since a recent update. @fweikert Don't worry about trying to fix that, just let me know...

@sgowroji Yes, you can go ahead, if syncing to head fixes the failing test. @alexei-grigoriev I'm guessing the test failure is unrelated, if it sticks around when moved to head...

Unfortunately, we made a change in the metrics code (replacing Duration with ints to save both memory and time), and now your change needs to be updated. I would suggest...

I agree that the estimates are not good in general and we need more ways to control them. Your PR looks reasonably good, but we'll have to consider if we...

@EdbertChan Any thoughts on my comment above?

ad 1: True, there is no forcing function, but there is the carrot that more accurate estimates would allow better use of local resources - faster builds when the estimate...

No, I don't believe this is a good approach. @wilwell is the person who works on this now, though.

Unresolved symlinks in tree artifacts are not currently being worked on, to the best of my knowledge.

I'm adding this to the docs in my CL removing `experimental_`.