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
v8-adapter project visibility & discovery #3
Comments
That's a really good point! I've been concerned about visibility for a while now, but was sort of letting it slide until there was a more complete feature set in the v8-adapter project. Answering your ideas in order:
|
Sounds good, I will prepare pull-request for
Concerning "feeling bad using one repository to advertise another" - that's true indeed, but I guess that this project could be really helpful for common users of J2V8 - thus it looks fair enough (at least from my point of view it's so, while there is nothing good like it around). Please, let me know if you want to rename the project to |
Hey @AlexTrotsenko ! I'm bringing this thread back to life... The official J2V8 repository has been silent, so I'm wondering if it may be better to branch off as a new project anyway. Thoughts? |
Indeed, j2v8 is almost not active since August 2017 - with only few minor commits in December. Something, which I found annoying already:
What is your thoughts on this topic? |
If we fork J2V8, I would probably fork it as a new project with a new name, and roll the features of this project into that one. My reasoning is that (I believe) the scope of J2V8 has grown wildly large, and it's trying to do more than provide a robust embeddable JS runtime...especially considering all the open issues and PRs. Specifically, if we were to fork that project, I would not want to support Node.js. I am sure many people like it, but the use cases that our organization needs do not include Node.js. From there, we could streamline the build process, and probably perform a number of performance enhancements to make the features of this project work better and with less hacks. |
@crahda what you are saying indeed make sense. I think, that Ideally "j2v8" should be core project with ability to extend it easily by projects like this one without any hacks. Concerning fork with another name: as for me, the main issue with maintaining such fork is merging useful changes from original repo to the fork. But taking into account, that unfortunately So, as for me, the question now is about time resources. E.g. At first glance, only re-structuring, clean-up, etc. will take significant amount of time. Thus in order to estimate efforts&time it makes sense for me to list upfront the goals, which are planned to be achieved with such re-structuring before stating the process. For example:
Some comments on last 2 items:
|
This is really good feedback, and I continue to believe that forking is probably the best option moving forward. The last update was in December of last year. A good roadmap will be important; our (Alicorn Systems') goal would be to create an embeddable Java Script run-time that can be cleanly extended to support modifications both at a high-level (e.g., in Java) and a low-level (e.g., in C++) without disrupting the core project. This kind of expandability is critical for a new project that we are working on that requires zero-copy data passing between Java and JS. To this end, and building off what you said, we would want to do the following (in order):
In all cases, if we assume responsibility for a fork of the J2V8, I would like to ensure that we begin writing our code to avoid as much garbage creation, blocking, and run-time reflection as possible. It will be difficult, but it could drastically improve the performance of the runtime in many cases (especially when dynamically injecting objects and classes). |
Do you have plans to do these? |
@windrunner414 Thanks for the note. I was planning to defer integration of the V8 project until it was definitively dead. We are about to hit the six-month mark, though, so I think it may be appropriate for us to fork. Let me work on some other issues and I will re-evaluate. |
If you're ready to start, we can work together |
Let's move this discussion to #17 . |
This project looks really great and provides nice way for sharing Java classes in JS runtime.
The only problem for me was - it is very hard to discover it.
Few ideas, which, as for me, might help other people to find, benefit and contribute to this project:
P.S. I have found this project when I was checking issues of J2V8 and thus found your comments about v8-adapter in the different issues. So, this is really helpful as well.
The text was updated successfully, but these errors were encountered: