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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stackoverflow in ovious_subtype due to splatting #39826

Closed
bienpierre opened this issue Feb 25, 2021 · 5 comments
Closed

Stackoverflow in ovious_subtype due to splatting #39826

bienpierre opened this issue Feb 25, 2021 · 5 comments

Comments

@bienpierre
Copy link

Hello,

I was expected an "error" syntax message with the following command, otherwise it filled the memory until system crashed or "interrupt execution" is proceed via htop or CTrL+C:
Capture d'écran du 2021-02-25 15 16 19

Regards

@bienpierre bienpierre changed the title Repl not displayed Error not displayed Feb 25, 2021
@simeonschaub
Copy link
Member

I don't see why this should error, since it's a valid thing to do, but we strongly discourage splatting large arrays, exactly because it causes excessive compile times.

@bienpierre
Copy link
Author

Ok I see...
I also tried this one. Is it also due to excessive compile time?
Capture d'écran du 2021-02-25 19 26 05
regards

@simeonschaub
Copy link
Member

Hmm, that does look like a more serious issue. I don't think this is related

@simeonschaub simeonschaub reopened this Feb 25, 2021
@simeonschaub simeonschaub changed the title Error not displayed Stackoverflow in ovious_subtype due to splatting Feb 25, 2021
@simeonschaub
Copy link
Member

I can reproduce on nightly and 1.5. I would not have expected a length of 2050 to produce a Stackoverflow in this example, but not 100% sure there is something to be done here.

@Keno
Copy link
Member

Keno commented Feb 25, 2021

Yeah, the system doesn't currently like tuples this long. Fixing that is #31681, so closing as a dup.

@Keno Keno closed this as completed Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants