Stu Hood

Results 228 comments of Stu Hood

> Besides, when env is dropped, all the completion queues are shutdown too, hence keeping a reference to channel is still useless. Then it sounds like the clone of the...

Using `filter_map` to completely recreate the graph can be much, much more efficient if your edge weights are cheaply cloneable.

@tdyas raised the point that the latter approach (a docker-aware intrinsic), adds the requirement that there is a docker image which corresponds to any remote execution environment, and some environments...

> @chrisjrn, I think it is reasonable to not support `` and `` when the environment is Docker or Remote. At least for now, until we get feedback from users...

> A major part of this ticket is deciding how to organize the sidebar of our docs. Another portion would be expanding and prioritizing a list of the potential howtos.

> Why didn't the engine make use of the rule providing `GoBuildContext` then? > > ``` > @rule > async def go_global_context(golang_subsystem: GolangSubsystem) -> GoBuildContext: > return GoBuildContext(cgo_allowed=golang_subsystem.cgo_allowed) > ```...

> It appears that log_cache_error is not working with tail tasks. https://github.com/pantsbuild/pants/actions/runs/3144144927/jobs/5110386492#step:13:598 > > I can add regular logging to show that the tail tasks are completing, but errors from...

> Btw, how do I add a category label? Is that a right given only to maintainers? [Contributors](https://www.pantsbuild.org/docs/the-pants-community#contributors) can as well: I'll ping you about that offline.

Thanks for the great description! @thejcannon is actually actively working on this one via #10271: see #16844.

I've added a section to https://docs.google.com/document/d/1vXRHWK7ZjAIp2BxWLwRYm1QOKDeXx02ONQWvXDloxkg/edit#heading=h.w0bkj2kpqzfq which describes how environment matchers should likely support this.