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

Implement tern server with Nashorn #33

Open
paulvi opened this issue Mar 28, 2014 · 6 comments
Open

Implement tern server with Nashorn #33

paulvi opened this issue Mar 28, 2014 · 6 comments

Comments

@paulvi
Copy link
Contributor

paulvi commented Mar 28, 2014

try and write results here

@angelozerr
Copy link
Owner

Please not that some tern plugins like closure, node.js depends on node.js for some features. Ex tern plugin node uses node to resolver module name declared inside require because it needs file system API)

@angelozerr angelozerr changed the title [research task] try to run tern with Nashorn jjs Implement tern server with Nashorn Nov 6, 2015
@angelozerr angelozerr added this to the 1.1.0 milestone Nov 6, 2015
angelozerr added a commit that referenced this issue Nov 6, 2015
@angelozerr
Copy link
Owner

@mickaelistria @vrubezhny @dgolovin @maxandersen @paulvi @fbricon @piotrtomiak I have implemented quickly tern server with Nashorn (but not the integration with Eclipse IDE).

You can find info at (samples to launch, etc) https://github.com/angelozerr/tern.java/blob/master/core/tern.server.nashorn/README.md

But I'm very disappointed, because I find the load of tern server is very slowly with Nashorn compared to Node.js or J2V8. Completion with big files with Nashorn is very slow -(

Perhaps my integration is not very performant?

@paulvi
Copy link
Contributor Author

paulvi commented Nov 7, 2015

At least, this is real world example for now.

BTW, https://github.com/angelozerr/tern.java/blob/master/README.md needs to be updated, it is not only idea now...

Showing that there's choice.

@angelozerr
Copy link
Owner

BTW, https://github.com/angelozerr/tern.java/blob/master/README.md needs to be updated, it is not only idea now...

done

@angelozerr angelozerr removed this from the 1.1.0 milestone Nov 9, 2015
@piotrtomiak
Copy link
Contributor

Perhaps my integration is not very performant?

No, it's not your fault ;) In the early days of adopting Tern in MyEclipse we've checked performance comparisons of Nashorn, Rhino and V8 (both Node.js and J2V8 are based on V8) and it looked like Nashorn is only slightly better than Rhino, and Rhino is pretty slow with Tern.

@angelozerr
Copy link
Owner

Thanks @piotrtomiak for your feedback. Yes today node > J2V8 > Nashorn > Rhino.

When you will have tipe, could you answer me for #363 with WebSocket topic. I have started to implement WebSocket on server/client side with Jetty (for the moment tern server send a message to the Java client and I display the result in the console). Any feedback are welcome. thanks!

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

No branches or pull requests

3 participants