Skip to content

Conversation

@rbuckton
Copy link
Contributor

The new awaited type is currently causing too many breaks to be considered stable for TS 3.9. This PR reverts the awaited type for this release while we continue to investigate the impact of the change.

Closes#37511
Closes#37512
Closes#37526
Closes#37528
Closes#37534
Closes#37551
Closes#37556
Closes#37564
Closes#37566
Closes#37567
Closes#37572
Closes#37592
Closes#37602
Closes#37603

@rbuckton
Copy link
ContributorAuthor

@typescript-bot perf test
@typescript-bot run dt
@typescript-bot test this
@typescript-bot user test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the perf test suite on this PR at a638aa8. You can monitor the build here.

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the parallelized Definitely Typed test suite on this PR at a638aa8. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the extended test suite on this PR at a638aa8. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the parallelized community code test suite on this PR at a638aa8. You can monitor the build here.

Copy link
Member

@sandersnsandersn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me.

@typescript-bot
Copy link
Collaborator

The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master.

@typescript-bot
Copy link
Collaborator

@rbuckton
The results of the perf run you requested are in!

Here they are:

Comparison Report - master..37610

Metricmaster37610DeltaBestWorst
Angular - node (v10.16.3, x64)
Memory used327,718k (± 0.02%)326,933k (± 0.04%)-786k (- 0.24%)326,512k327,111k
Parse Time1.63s (± 0.58%)1.61s (± 0.51%)-0.01s (- 0.80%)1.60s1.64s
Bind Time0.89s (± 1.01%)0.88s (± 0.96%)-0.00s (- 0.56%)0.86s0.90s
Check Time4.78s (± 0.48%)4.75s (± 0.51%)-0.03s (- 0.57%)4.68s4.81s
Emit Time5.32s (± 0.88%)5.31s (± 0.47%)-0.02s (- 0.30%)5.27s5.37s
Total Time12.61s (± 0.55%)12.55s (± 0.35%)-0.06s (- 0.50%)12.44s12.62s
Monaco - node (v10.16.3, x64)
Memory used327,135k (± 0.02%)327,078k (± 0.02%)-57k (- 0.02%)326,936k327,296k
Parse Time1.25s (± 0.61%)1.25s (± 0.29%)-0.00s (- 0.08%)1.25s1.26s
Bind Time0.77s (± 0.58%)0.77s (± 0.44%)+0.00s (+ 0.13%)0.77s0.78s
Check Time4.73s (± 0.26%)4.74s (± 0.62%)+0.01s (+ 0.11%)4.67s4.80s
Emit Time2.92s (± 0.71%)2.91s (± 0.50%)-0.01s (- 0.48%)2.87s2.94s
Total Time9.69s (± 0.22%)9.68s (± 0.36%)-0.01s (- 0.10%)9.61s9.75s
TFS - node (v10.16.3, x64)
Memory used291,963k (± 0.02%)291,993k (± 0.03%)+30k (+ 0.01%)291,878k292,172k
Parse Time0.95s (± 0.47%)0.95s (± 0.38%)-0.01s (- 0.63%)0.94s0.95s
Bind Time0.75s (± 0.80%)0.74s (± 0.66%)-0.00s (- 0.27%)0.73s0.75s
Check Time4.26s (± 0.58%)4.26s (± 0.29%)+0.00s (+ 0.05%)4.24s4.28s
Emit Time3.06s (± 0.36%)3.05s (± 0.80%)-0.01s (- 0.46%)3.00s3.09s
Total Time9.02s (± 0.28%)9.00s (± 0.27%)-0.01s (- 0.16%)8.94s9.05s
material-ui - node (v10.16.3, x64)
Memory used452,881k (± 0.01%)452,579k (± 0.01%)-302k (- 0.07%)452,488k452,666k
Parse Time1.77s (± 0.31%)1.77s (± 0.37%)0.00s ( 0.00%)1.76s1.79s
Bind Time0.68s (± 0.77%)0.68s (± 0.59%)0.00s ( 0.00%)0.67s0.69s
Check Time13.66s (± 0.69%)13.60s (± 0.56%)-0.06s (- 0.43%)13.42s13.76s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time16.11s (± 0.59%)16.06s (± 0.46%)-0.06s (- 0.37%)15.88s16.21s
Angular - node (v12.1.0, x64)
Memory used303,414k (± 0.03%)302,662k (± 0.08%)-752k (- 0.25%)301,675k302,946k
Parse Time1.58s (± 0.56%)1.57s (± 0.41%)-0.01s (- 0.32%)1.56s1.59s
Bind Time0.87s (± 0.83%)0.87s (± 0.68%)+0.00s (+ 0.23%)0.86s0.89s
Check Time4.64s (± 0.45%)4.64s (± 1.05%)+0.00s (+ 0.02%)4.58s4.83s
Emit Time5.47s (± 0.46%)5.48s (± 1.00%)+0.01s (+ 0.13%)5.39s5.67s
Total Time12.56s (± 0.26%)12.57s (± 0.78%)+0.00s (+ 0.04%)12.46s12.94s
Monaco - node (v12.1.0, x64)
Memory used307,099k (± 0.02%)307,015k (± 0.02%)-83k (- 0.03%)306,795k307,128k
Parse Time1.20s (± 0.62%)1.20s (± 0.49%)+0.00s (+ 0.33%)1.19s1.22s
Bind Time0.75s (± 1.02%)0.74s (± 0.92%)-0.01s (- 1.21%)0.73s0.76s
Check Time4.56s (± 0.22%)4.55s (± 0.45%)-0.00s (- 0.04%)4.53s4.61s
Emit Time2.96s (± 0.49%)2.95s (± 0.47%)-0.01s (- 0.30%)2.92s2.97s
Total Time9.46s (± 0.20%)9.45s (± 0.28%)-0.01s (- 0.13%)9.38s9.49s
TFS - node (v12.1.0, x64)
Memory used274,292k (± 0.03%)274,307k (± 0.02%)+15k (+ 0.01%)274,166k274,404k
Parse Time0.93s (± 0.51%)0.93s (± 0.48%)-0.00s (- 0.11%)0.92s0.94s
Bind Time0.70s (± 1.23%)0.70s (± 1.18%)+0.00s (+ 0.43%)0.69s0.73s
Check Time4.17s (± 0.43%)4.19s (± 0.48%)+0.02s (+ 0.41%)4.14s4.23s
Emit Time3.10s (± 1.10%)3.09s (± 0.99%)-0.01s (- 0.45%)3.02s3.16s
Total Time8.91s (± 0.43%)8.91s (± 0.47%)+0.00s (+ 0.01%)8.84s9.01s
material-ui - node (v12.1.0, x64)
Memory used430,291k (± 0.01%)429,928k (± 0.05%)-364k (- 0.08%)429,033k430,241k
Parse Time1.75s (± 0.40%)1.75s (± 0.33%)+0.00s (+ 0.06%)1.74s1.76s
Bind Time0.63s (± 0.75%)0.63s (± 1.02%)+0.00s (+ 0.32%)0.62s0.65s
Check Time12.15s (± 0.55%)12.10s (± 0.29%)-0.05s (- 0.41%)12.04s12.20s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time14.53s (± 0.42%)14.48s (± 0.26%)-0.05s (- 0.35%)14.40s14.58s
Angular - node (v8.9.0, x64)
Memory used322,812k (± 0.02%)322,061k (± 0.01%)-752k (- 0.23%)321,957k322,166k
Parse Time2.12s (± 0.21%)2.10s (± 0.33%)-0.01s (- 0.61%)2.09s2.12s
Bind Time0.91s (± 0.37%)0.92s (± 0.79%)+0.00s (+ 0.55%)0.91s0.94s
Check Time5.48s (± 1.18%)5.43s (± 1.06%)-0.05s (- 0.91%)5.26s5.51s
Emit Time6.24s (± 1.95%)6.26s (± 1.28%)+0.02s (+ 0.40%)6.14s6.54s
Total Time14.75s (± 0.72%)14.71s (± 0.33%)-0.03s (- 0.22%)14.59s14.85s
Monaco - node (v8.9.0, x64)
Memory used325,482k (± 0.01%)325,498k (± 0.01%)+16k (+ 0.00%)325,424k325,593k
Parse Time1.55s (± 0.44%)1.54s (± 0.43%)-0.00s (- 0.19%)1.53s1.56s
Bind Time0.89s (± 0.38%)0.89s (± 0.58%)-0.00s (- 0.45%)0.88s0.90s
Check Time5.35s (± 0.65%)5.37s (± 0.43%)+0.01s (+ 0.21%)5.32s5.42s
Emit Time3.51s (± 0.72%)3.50s (± 0.36%)-0.01s (- 0.31%)3.48s3.53s
Total Time11.30s (± 0.47%)11.30s (± 0.24%)-0.00s (- 0.03%)11.25s11.35s
TFS - node (v8.9.0, x64)
Memory used291,358k (± 0.02%)291,365k (± 0.01%)+7k (+ 0.00%)291,275k291,419k
Parse Time1.25s (± 0.47%)1.26s (± 0.47%)+0.00s (+ 0.16%)1.24s1.27s
Bind Time0.75s (± 0.64%)0.74s (± 0.63%)-0.01s (- 0.94%)0.73s0.75s
Check Time4.88s (± 1.27%)4.86s (± 1.90%)-0.02s (- 0.47%)4.74s5.12s
Emit Time3.31s (± 2.11%)3.29s (± 1.51%)-0.02s (- 0.48%)3.15s3.36s
Total Time10.19s (± 0.39%)10.15s (± 0.51%)-0.04s (- 0.39%)10.05s10.31s
material-ui - node (v8.9.0, x64)
Memory used455,455k (± 0.01%)455,141k (± 0.01%)-313k (- 0.07%)455,022k455,220k
Parse Time2.11s (± 0.72%)2.10s (± 0.53%)-0.00s (- 0.14%)2.08s2.13s
Bind Time0.81s (± 1.03%)0.81s (± 1.19%)+0.00s (+ 0.37%)0.79s0.83s
Check Time17.64s (± 0.62%)17.65s (± 0.38%)+0.01s (+ 0.09%)17.52s17.83s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time20.55s (± 0.59%)20.56s (± 0.36%)+0.02s (+ 0.08%)20.43s20.77s
Angular - node (v8.9.0, x86)
Memory used185,749k (± 0.02%)185,365k (± 0.03%)-385k (- 0.21%)185,244k185,519k
Parse Time2.05s (± 0.35%)2.06s (± 0.84%)+0.01s (+ 0.24%)2.03s2.11s
Bind Time1.07s (± 0.52%)1.07s (± 0.35%)-0.00s (- 0.28%)1.06s1.07s
Check Time4.99s (± 0.59%)4.99s (± 0.32%)-0.00s (- 0.02%)4.95s5.03s
Emit Time6.06s (± 0.44%)6.03s (± 0.68%)-0.03s (- 0.49%)5.97s6.14s
Total Time14.17s (± 0.35%)14.15s (± 0.45%)-0.02s (- 0.17%)14.04s14.31s
Monaco - node (v8.9.0, x86)
Memory used185,276k (± 0.02%)185,251k (± 0.02%)-24k (- 0.01%)185,185k185,330k
Parse Time1.59s (± 0.76%)1.61s (± 1.19%)+0.01s (+ 0.75%)1.57s1.66s
Bind Time0.76s (± 0.58%)0.77s (± 1.22%)+0.01s (+ 1.05%)0.75s0.79s
Check Time5.39s (± 0.44%)5.41s (± 0.68%)+0.02s (+ 0.32%)5.34s5.49s
Emit Time2.86s (± 0.83%)2.89s (± 1.28%)+0.03s (+ 1.05%)2.81s3.00s
Total Time10.61s (± 0.42%)10.68s (± 0.76%)+0.07s (+ 0.69%)10.53s10.91s
TFS - node (v8.9.0, x86)
Memory used166,797k (± 0.02%)166,763k (± 0.02%)-34k (- 0.02%)166,687k166,877k
Parse Time1.29s (± 0.60%)1.28s (± 0.38%)-0.01s (- 0.39%)1.27s1.29s
Bind Time0.71s (± 0.70%)0.71s (± 0.81%)-0.01s (- 0.70%)0.70s0.72s
Check Time4.61s (± 0.64%)4.60s (± 0.54%)-0.01s (- 0.20%)4.55s4.66s
Emit Time2.99s (± 1.06%)2.96s (± 1.08%)-0.03s (- 1.14%)2.89s3.01s
Total Time9.60s (± 0.44%)9.55s (± 0.44%)-0.05s (- 0.54%)9.46s9.63s
material-ui - node (v8.9.0, x86)
Memory used257,642k (± 0.02%)257,423k (± 0.01%)-219k (- 0.08%)257,369k257,517k
Parse Time2.19s (± 0.70%)2.18s (± 0.57%)-0.01s (- 0.32%)2.14s2.19s
Bind Time0.68s (± 1.37%)0.68s (± 1.20%)+0.00s (+ 0.29%)0.66s0.70s
Check Time16.26s (± 0.87%)16.19s (± 0.44%)-0.07s (- 0.44%)16.10s16.41s
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)0.00s ( NaN%)0.00s0.00s
Total Time19.12s (± 0.75%)19.05s (± 0.40%)-0.07s (- 0.39%)18.96s19.30s
System
Machine Namets-ci-ubuntu
Platformlinux 4.4.0-166-generic
Architecturex64
Available Memory16 GB
Available Memory1 GB
CPUs4 × Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
Hosts
  • node (v10.16.3, x64)
  • node (v12.1.0, x64)
  • node (v8.9.0, x64)
  • node (v8.9.0, x86)
Scenarios
  • Angular - node (v10.16.3, x64)
  • Angular - node (v12.1.0, x64)
  • Angular - node (v8.9.0, x64)
  • Angular - node (v8.9.0, x86)
  • Monaco - node (v10.16.3, x64)
  • Monaco - node (v12.1.0, x64)
  • Monaco - node (v8.9.0, x64)
  • Monaco - node (v8.9.0, x86)
  • TFS - node (v10.16.3, x64)
  • TFS - node (v12.1.0, x64)
  • TFS - node (v8.9.0, x64)
  • TFS - node (v8.9.0, x86)
  • material-ui - node (v10.16.3, x64)
  • material-ui - node (v12.1.0, x64)
  • material-ui - node (v8.9.0, x64)
  • material-ui - node (v8.9.0, x86)
BenchmarkNameIterations
Current3761010
Baselinemaster10

@rbuckton
Copy link
ContributorAuthor

I found a few more stragglers, will update shortly.

@rbuckton
Copy link
ContributorAuthor

@typescript-bot run dt
@typescript-bot test this
@typescript-bot user test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the extended test suite on this PR at 28946c3. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the parallelized community code test suite on this PR at 28946c3. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 25, 2020

Heya @rbuckton, I've started to run the parallelized Definitely Typed test suite on this PR at 28946c3. You can monitor the build here.

@rbuckton
Copy link
ContributorAuthor

@typescript-bot run dt
@typescript-bot test this
@typescript-bot user test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 26, 2020

Heya @rbuckton, I've started to run the parallelized Definitely Typed test suite on this PR at 5017cb8. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 26, 2020

Heya @rbuckton, I've started to run the parallelized community code test suite on this PR at 5017cb8. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 26, 2020

Heya @rbuckton, I've started to run the extended test suite on this PR at 5017cb8. You can monitor the build here.

@rbuckton
Copy link
ContributorAuthor

@rbucktonrbuckton merged commit 4fc4c4e into masterMar 26, 2020
@rbuckton
Copy link
ContributorAuthor

@typescript-bot cherry-pick this to release-3.9

@typescript-bot
Copy link
Collaborator

typescript-bot commented Mar 26, 2020

Heya @rbuckton, I've started to run the task to cherry-pick this into release-3.9 on this PR at 5017cb8. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

Hey @rbuckton, I've opened #37614 for you.

@Luxcium
Copy link

oops... I was relying on this solution because I have a Promise<MaybeList<Promise<ISymbolSearchResult[]>>>
where MaybeList is an array abstraction so I am using promise.all to remove the inner Promise I don't know what to do now to avoid using as unknown as MaybeList<ISymbolSearchResult[]> inside of my async function (which should be returning the Promise<MaybeList<ISymbolSearchResult[]>> instead of the Promise<MaybeList<Promise<ISymbolSearchResult[]>>>)

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.