#general
- b
bored-art-40741
09/08/2015, 10:19 PMBut it's noticeably slower - b
bored-art-40741
09/08/2015, 10:20 PMIt hovers at 8gb heap, burns up to 10gb, does a long STW to get back down to 8gb, rinse and repeat - w
witty-crayon-22786
09/08/2015, 10:22 PM(so, note that switching GCs causing this issue to go away does not necessarily mean there isn't a leak, but) - b
bored-art-40741
09/08/2015, 10:22 PMYeah - b
bored-art-40741
09/08/2015, 10:35 PMMaybe I'm overthinking this - b
bored-art-40741
09/08/2015, 10:35 PMI did confirm that on one particular target we had about 1500 dependencies on the classpath - b
bored-art-40741
09/08/2015, 10:36 PMWhich means at least 1500 analysis files loaded into memory - b
bored-art-40741
09/08/2015, 10:36 PMIn addition to whatever else is being done in parallel - b
bored-art-40741
09/08/2015, 10:36 PMAnd G1 seems much more willing to GC old gen on the small targets - b
bored-art-40741
09/08/2015, 10:36 PMWe'll see how it does when it gets to a larger one - b
bored-art-40741
09/08/2015, 10:37 PMDoesn't feel like it's getting quite the same throughput as before, but it also isn't spending much time at all in GC - b
bored-art-40741
09/08/2015, 10:38 PM1-2% - b
bored-art-40741
09/08/2015, 10:39 PMwhooooa - b
bored-art-40741
09/08/2015, 10:39 PMscalac just blew the stack - b
bored-art-40741
09/08/2015, 10:39 PMIn some sort of crazy implicit lookup loop - b
bored-art-40741
09/08/2015, 10:39 PMOh maybe I deleted the xss we were passing before - d
dry-analyst-73584
09/08/2015, 10:54 PM@enough-analyst-54434: @ancient-stone-50795 I’ve responded to the comments and fixed a couple of things. This is now ready for review: https://rbcommons.com/s/twitter/r/2755/ - d
dry-analyst-73584
09/08/2015, 11:11 PMCould someone please assign this issue to me? https://github.com/pantsbuild/pants/issues/2049 - e
enough-analyst-54434
09/08/2015, 11:12 PMJust a sec - you need committer perms to do useful things with issues, but we have a contributor role that is the next best we can do via the github interface - e
enough-analyst-54434
09/08/2015, 11:13 PMOh wait - we did this dance, sorry - short memory - e
enough-analyst-54434
09/08/2015, 11:13 PMAssigned - and thank you! - d
dry-analyst-73584
09/08/2015, 11:14 PMThanks simple smile - b
bored-art-40741
09/08/2015, 11:23 PM@witty-crayon-22786: G1 seems to do the trick - b
bored-art-40741
09/08/2015, 11:23 PMThe heap is much better behaved - w
witty-crayon-22786
09/08/2015, 11:23 PMbetter behaviour than CMS? - w
witty-crayon-22786
09/08/2015, 11:23 PMneat - b
bored-art-40741
09/08/2015, 11:23 PMYeah - b
bored-art-40741
09/08/2015, 11:23 PMVery little time spent in GC, too - w
witty-crayon-22786
09/08/2015, 11:23 PMwe're not all the way on 8 yet, but that's very good to know. - b
bored-art-40741
09/08/2015, 11:23 PMAbout 1-2%