Create a gist now

Instantly share code, notes, and snippets.

What would you like to do?
Scala incremental compiler improvements by Grzegorz Kossakowski. Actual runtimes are not so important here, since the two benchmarks were run on different machines, and also they make no attempt to ensure that the scala compiler is fully JITed, which during normal Play development offers significant improvements over these times as the compiler …
Initial clean compile of entire project:
[info] [info] Compiling 29 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 15 s, completed Aug 28, 2013 9:43:38 PM
Now simulate adding a method to a controller:
[info] [info] Compiling 1 Scala source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 2 s, completed Aug 28, 2013 9:43:40 PM
Now simulate adding a new route to the routes file:
[info] [info] Compiling 2 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 5 s, completed Aug 28, 2013 9:43:45 PM
Initial clean compile of entire project:
[info] [info] Compiling 29 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 25 s, completed 29/08/2013 2:49:17 PM
Now simulate adding a method to a controller:
[info] [info] Compiling 1 Scala source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [info] Compiling 29 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 16 s, completed 29/08/2013 2:49:32 PM
Now simulate adding a new route to the routes file:
[info] [info] Compiling 2 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [info] Compiling 29 Scala sources and 1 Java source to /tmp/inc-compile/target/scala-2.10/classes...
[info] [success] Total time: 10 s, completed 29/08/2013 2:49:43 PM
@diwa-zz

This comment has been minimized.

Show comment Hide comment
@diwa-zz

diwa-zz Aug 29, 2013

Excellent news. How/When do we get to enjoy this in play?

diwa-zz commented Aug 29, 2013

Excellent news. How/When do we get to enjoy this in play?

@gkossakowski

This comment has been minimized.

Show comment Hide comment
@gkossakowski

gkossakowski Aug 29, 2013

@Diwa: I hope I can gets this merged in subsequent releases of sbt 0.13.x.

@Diwa: I hope I can gets this merged in subsequent releases of sbt 0.13.x.

@diwa-zz

This comment has been minimized.

Show comment Hide comment
@diwa-zz

diwa-zz Aug 30, 2013

That's great. Looking forward eagerly.

diwa-zz commented Aug 30, 2013

That's great. Looking forward eagerly.

@ms-tg

This comment has been minimized.

Show comment Hide comment
@ms-tg

ms-tg Sep 4, 2013

Fantastic!

ms-tg commented Sep 4, 2013

Fantastic!

@benmccann

This comment has been minimized.

Show comment Hide comment
@benmccann

benmccann Sep 6, 2013

This is so awesome. I cannot wait!!!

Are the changes the pull request below or are there more pull requests to come?
sbt/sbt#839

This is so awesome. I cannot wait!!!

Are the changes the pull request below or are there more pull requests to come?
sbt/sbt#839

@magro

This comment has been minimized.

Show comment Hide comment
@magro

magro Sep 14, 2013

+1, awesome!

magro commented Sep 14, 2013

+1, awesome!

@peter-fu

This comment has been minimized.

Show comment Hide comment
@peter-fu

peter-fu Sep 22, 2013

+1, looking forward to it!

+1, looking forward to it!

@gkossakowski

This comment has been minimized.

Show comment Hide comment
@gkossakowski

gkossakowski Nov 7, 2013

FYI: I started merging my changes back to sbt. It will take a few weeks to finish.

You can follow my work here:

FYI: I started merging my changes back to sbt. It will take a few weeks to finish.

You can follow my work here:

@rpokrywka

This comment has been minimized.

Show comment Hide comment
@rpokrywka

rpokrywka Nov 11, 2013

Cool

Cool

@gkossakowski

This comment has been minimized.

Show comment Hide comment
@gkossakowski

gkossakowski Nov 27, 2013

I decided to make it easier to follow the status of the merge by creating a meta issue. Check it out here: sbt/sbt#1010

I decided to make it easier to follow the status of the merge by creating a meta issue. Check it out here: sbt/sbt#1010

@hedefalk

This comment has been minimized.

Show comment Hide comment
@hedefalk

hedefalk Nov 27, 2013

Awesome!

Awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment