I have investigated this myself and you don't need to pop environmentals. Tapestry cleans up any thread locals at the end of a request so that a new request will not get dirty values.
Obviously it's best to clean up any environmentals when they are no longer required but sometimes that is not possible.
