[release/7.0-rc1] Improve performance of Tar library #74305
Conversation
Not a perf thing, just readability.
…sync variants, and overhaul GenerateExtendedAttributesDataStream
|
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsBackport of #74281 to release/7.0-rc1 /cc @carlossanlop @stephentoub Customer ImpactTestingRiskIMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.
|
|
This change is relatively large and mostly optional (if current performance is considered shippable). As last time, we will have essentially no time to react to RC1 feedback before GA. Is there a subset (including the functional fixes) that would get most of the benefit with less risk of introducing a bug? I'm not necessarily opposed to taking it all, if the risk/value is there. But polishing a new feature does not meet the bar in itself. |
|
I'm going to close this per above. It is too late for this kind of change. If there are functional fixes in here that meet the bar please extract those to port them. |
ghost
locked as resolved and limited conversation to collaborators
Backport of #74281 to release/7.0-rc1
/cc @carlossanlop @stephentoub
Customer Impact
This is a new feature for 7.0. Customer impact should be minimal but they will get considerable performance improvements. See description in original PR.
Testing
The CI passed.
Risk
Low, new feature in 7.0.
IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.