Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tracking Issue: Splitting Types in NodeJS #11558

Open
1 of 6 tasks
RobbieMcKinstry opened this issue Dec 6, 2022 · 1 comment
Open
1 of 6 tasks

Tracking Issue: Splitting Types in NodeJS #11558

RobbieMcKinstry opened this issue Dec 6, 2022 · 1 comment
Labels
area/codegen SDK-gen, program-gen, convert impact/performance Something is slower than expected kind/enhancement Improvements or new features language/javascript size/L Estimated effort to complete (up to 10 days).

Comments

@RobbieMcKinstry
Copy link
Contributor

RobbieMcKinstry commented Dec 6, 2022

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

This issue tracks multiple issues related to splitting files in the /types directory into smaller units. Originally, this work was completed in #10831 but reverted in #11534 due to a bug in TSC. The issues below are listed in priority order.

Affected area/feature

NodeJS code gen

Related Issues

@RobbieMcKinstry RobbieMcKinstry added kind/enhancement Improvements or new features impact/performance Something is slower than expected language/javascript area/codegen SDK-gen, program-gen, convert size/L Estimated effort to complete (up to 10 days). labels Dec 6, 2022
@RobbieMcKinstry RobbieMcKinstry self-assigned this Dec 6, 2022
@Navigatron
Copy link

A big +1 for this. Using the azure-native provider and creating just three resources, we're seeing:

[xxx]$ NODE_OPTIONS=--max-old-space-size=6000 tsc --extendedDiagnostics
Files:                        30888
Lines of Library:             29152
Lines of Definitions:       5098802
Lines of TypeScript:            240
Lines of JavaScript:              0
Lines of JSON:                    0
Lines of Other:                   0
Identifiers:                4024654
Symbols:                    1974508
Types:                           88
Instantiations:                   0
Memory used:               3475317K
Assignability cache size:         0
Identity cache size:              0
Subtype cache size:               0
Strict subtype cache size:        0
I/O Read time:               15.91s
Parse time:                  27.31s
ResolveModule time:           5.52s
ResolveTypeReference time:    0.09s
Program time:                56.65s
Bind time:                   72.25s
Total time:                 128.90s

Also related:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/codegen SDK-gen, program-gen, convert impact/performance Something is slower than expected kind/enhancement Improvements or new features language/javascript size/L Estimated effort to complete (up to 10 days).
Projects
None yet
Development

No branches or pull requests

2 participants