Issue 5023 in pharo: "senders of" is unusable for messages send often

Status: Accepted
Owner: [hidden email]
CC: [hidden email],  [hidden email]
Labels: Type-Bug Milestone-1.4 Importance-High

New issue 5023 by [hidden email]: "senders of" is unusable for  
messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

try to find all the senders of and:

   --> takes multiple minutes to show up
   --> is unusably slow



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #1 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

really minutes?
this is true that running a profiler can help to see if we can optimize it.
Stef


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

Yes, i confirm. Almost instantaneous in Pharo 1.3 and very slow in 1.4.  
Something strange happened between.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #3 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

Issues 5038 and 5039 should help.

Maybe deaper investigation is needed


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #4 on issue 5023 by [hidden email]: "senders of" is unusable for  
messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

Aren't selectors which were previously inlined now kept as (non-used)  
literals in the compiledMethods?
This is so they actually show up at all...

So the reason it's slow in 1.4, would probably be that you actually get  
accurate results.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #5 on issue 5023 by [hidden email]: "senders of" is unusable for  
messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

Doesn't mean the actual process couldn't possibly be optimized to scale  
better ofc, but it's probably the main reason it's slower, not some change  
in the implementation itself which could be reverted.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #6 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

No, I played with the ifTrue: "senders of" when that change was done. It  
was fast.

This is something else...


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #7 on issue 5023 by marianopeck: "senders of" is unusable for  
messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

Implementors is also terrible slow...I opened an issue for that some time  
ago.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #8 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

It's the same UI.

I optimized it a lot today :)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Duplicate
        Mergedinto: 5006

Comment #9 on issue 5023 by [hidden email]: "senders of" is unusable  
for messages send often
http://code.google.com/p/pharo/issues/detail?id=5023

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker