search

puniverse / pulsar

Fibers, Channels and Actors for Clojure
http://docs.paralleluniverse.co/pulsar/
Other
911 stars 53 forks source link

java 8 support #10

Closed trevor closed 10 years ago

trevor commented 11 years ago

Using this with Java 8 generates numerous "[quasar] ERROR: Unable to instrument ...". Does anyone know how to enable Java 8 support?

pubot commented 11 years ago

I'll take a look at it in a week or so. On Aug 26, 2013 6:55 PM, "Trevor Wennblom" notifications@github.com wrote:

Using this with Java 8 generates numerous "[quasar] ERROR: Unable to instrument ...". Does anyone know how to enable Java 8 support?

— Reply to this email directly or view it on GitHubhttps://github.com/puniverse/pulsar/issues/10 .

trevor commented 11 years ago

Great, thank you!

pron commented 11 years ago

This might take a little longer. Not that it's hard – it's probably trivial, but we're swamped with other (really exciting) stuff regarding Quasar at the moment.

trevor commented 11 years ago

@pron FWIW just wanted to mention we're still interested in this. Thanks!

pron commented 11 years ago

@trevor Rest assured, so are we! To keep you updated, I'm currently working on improving Quasar (better isolation, time-slice preemption, hot code loading and more) and on a demo. Once the demo is done (as well as some of the aforementioned features), I'll take a look at Java 8. The main reason we're pushing this particular feature back is that Java 8 is still very much a moving target, especially when it comes to java.util.concurrent, which is at the very core of Quasar/Pulsar.

Java 8 isn't feature complete yet, and it's certainly not production ready, and Quasar uses very deep, low-level code. It might be a very easy integration, and it might not, but we don't want to be fighting JDK bugs. I'll give it a try when I have less-pressing work, and if it's not trivial, we'll do it when Java 8 matures a bit.

pron commented 10 years ago

Development version (0.5.0-SNAPSHOT) now supports Java 8. It turns out all that was required was a new ASM version (5.0_BETA).