Azure / azure-sdk-for-js

This repository is for active development of the Azure SDK for JavaScript (NodeJS & Browser). For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/javascript/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-js.
MIT License
2.09k stars 1.2k forks source link

[build] sdk/network/arm-network-rest takes more than 1 minute to build #27596

Open jeremymeng opened 1 year ago

jeremymeng commented 1 year ago

This file is causing problem:

https://github.com/Azure/azure-sdk-for-js/blob/5ec3a86fa3ac4616f949802eb7206f20254d2db0/sdk/network/arm-network-rest/src/isUnexpected.ts

image

kazrael2119 commented 1 year ago

@jeremymeng ,I can't repro this issue, could you give some backgrounds of this issue? thanks

jeremymeng commented 1 year ago

I found out that it is not related to tests or test proxy, but TypeScript itself took a lot of time to build the source file isUnexpected.ts. Not sure whether certain pattern in that particular file caused the compiler to be so slow. But it would be nice if we could investigate and see if we can improve the generated

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3245539&view=logs&j=f3d84248-526b-5c53-c1d9-c908a2c076cb&t=a08660e7-0c74-50b6-45ab-0cb70a5776f5&l=464 The log shows that it took > 2 minutes to run the test. image

jeremymeng commented 11 months ago

arm-network slow to build too image