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

Potential issue with is default and Promises #6621

Open
p6rt opened this issue Oct 20, 2017 · 2 comments
Open

Potential issue with is default and Promises #6621

p6rt opened this issue Oct 20, 2017 · 2 comments

Comments

@p6rt
Copy link

p6rt commented Oct 20, 2017

Migrated from rt.perl.org#132326 (status was 'new')

Searchable as RT132326$

@p6rt
Copy link
Author

p6rt commented Oct 20, 2017

From @zoffixznet

Today we had to revert an `is default` use in Promise.pm​: rakudo/rakudo@d20e972a12

The commit caused a reproducible hang in S17-promise/lock-async-stress.t and S17-promise/lock-async-stress2.t
tests while running stresstests, although when running the test files on their own, they passed (each takes ~50s to run)

It's possible there's some issue in `is default` that occurs under stress.

IRC​: https://irclog.perlgeek.de/perl6-dev/2017-10-20#i_15330009

@p6rt
Copy link
Author

p6rt commented Oct 20, 2017

From @zoffixznet

https://irclog.perlgeek.de/perl6-dev/2017-10-20#i_15330070

13​:29 jnthn It'll probably be lazy attribute initialization
13​:30 lizmat jnthn​: but "is default" init isn't lazy ?
13​:30 is it?
13​:31 hmmm
13​:31 jnthn Attribute scalar container creation is
13​:31 Down in The Guts
13​:31 But I'd like to kill that off
13​:31 It makes every attribute access more expensive
13​:32 lizmat so how would you like to fix that ?
13​:32 jnthn I don't know yet.

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

1 participant