-
Notifications
You must be signed in to change notification settings - Fork 42
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
define a default js engine #275
Labels
Milestone
Comments
davidcoutadeur
pushed a commit
that referenced
this issue
Apr 12, 2024
davidcoutadeur
pushed a commit
that referenced
this issue
Apr 12, 2024
xavierba
pushed a commit
to xavierba/lsc
that referenced
this issue
Apr 12, 2024
xavierba
pushed a commit
to xavierba/lsc
that referenced
this issue
Apr 12, 2024
xavierba
pushed a commit
to xavierba/lsc
that referenced
this issue
Apr 12, 2024
xavierba
pushed a commit
to xavierba/lsc
that referenced
this issue
Apr 12, 2024
xavierba
pushed a commit
to xavierba/lsc
that referenced
this issue
Apr 15, 2024
@davidcoutadeur does the team have an idea when 2.2 would be released? |
The team is working on the last issues. As this is a community project, we can't promise to release at a well-defined date. Though, there are not much issues for finishing the release (66% of issues closed) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, there are multiple ways to use javascript in dataset/pivot transformation/conditions.
In next version 2.2, we'll have the new graalvm engine.
See also:
Multiple issues here:
js:
is present). I suppose it is graalvm. We should document this, and especially in https://lsc-project.org/documentation/latest/scripting.htmlFor example, in lsc.xml, we sometimes have dataset like these:
But even if
js:
is specified here, the graaljs engine seems used for evaluation (as shown in the logs below)We should clarify in documentation:
js:
is not supporting the java object anymore in java 17. This leads to errors like this one:Removing
js:
before each script in the lsc.xml used for test solves the issue here. Using explicitelygj:
also solves the issue.I'll push a PR for fixing the tests.
@artlog: as you have worked on graaljs engine, if you have some infos, don't hesite to share here.
The text was updated successfully, but these errors were encountered: