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

Java Generation: Generate classes for concrete function parameters #2783

Merged
merged 7 commits into from May 13, 2024

Conversation

aziemchawdhary-gs
Copy link
Contributor

@aziemchawdhary-gs aziemchawdhary-gs commented Apr 16, 2024

M2M execution relies on source fetch tree computation to generate the list of Pure classes to generate Java code for.
However, if a function is passed an empty argument then there is no property access and hence nothing is present in the source tree informing the platform that Java code should be generated for the corresponding parameter type. As a result the compilation of Java code present in the execution plan fails due to missing classes.

This change fixes that by processing the parameters and generating code for parameter types that are not part of the Java standard library or engine conventions library. If we happen to have already generated code for the parameter type earlier in the plan generation (e.g the class is mentioned in the source fetch tree) then the Java generation merging code will handle duplicates that may be generated by this changeset.

@aziemchawdhary-gs aziemchawdhary-gs requested a review from a team as a code owner April 16, 2024 13:49
@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as draft April 16, 2024 14:36
Copy link

github-actions bot commented Apr 16, 2024

Test Results

     763 files  +  2       763 suites  +2   1h 6m 35s ⏱️ - 1m 46s
12 598 tests +49  12 365 ✔️ +49  233 💤 ±0  0 ±0 
15 709 runs  +49  15 458 ✔️ +49  251 💤 ±0  0 ±0 

Results for commit 9a8cd94. ± Comparison against base commit d7d0f47.

♻️ This comment has been updated with latest results.

@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as ready for review April 17, 2024 09:40
@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as draft April 17, 2024 14:53
@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as ready for review April 17, 2024 17:46
@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as draft April 19, 2024 10:27
@aziemchawdhary-gs aziemchawdhary-gs marked this pull request as ready for review May 7, 2024 21:53
@davidharte-gs davidharte-gs self-requested a review May 10, 2024 22:25
@aziemchawdhary-gs aziemchawdhary-gs merged commit 3b4d5b6 into finos:master May 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants