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.

Sign up for GitHub

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

Jump to bottom

Use new CTPG parser in Java VM of the UDF client #969

Closed
tomuben opened this issue Oct 1, 2024 · 0 comments · Fixed by #970
Closed

Use new CTPG parser in Java VM of the UDF client #969

tomuben opened this issue Oct 1, 2024 · 0 comments · Fixed by #970
Assignees
@tomuben
Labels
refactoring Code improvement without behavior change

Comments

@tomuben
Copy link
Collaborator

tomuben commented Oct 1, 2024 •
edited
Loading

Background

Part of #652
In #967 we added the new parser. Now we need to use it in the Java VM

Acceptance Criteria

  1. Add a new implementation of the Jave Options Parser which uses the new CTPG implementation, which implements the interface in https://github.com/exasol/script-languages/blob/master/exaudfclient/base/javacontainer/script_options/parser.h
  2. Add unit tests
  3. Add a parameter to the Java VM

What is not scope of this Task

  1. Do not modify the UDF client to pass the parameter for using the new parser. The UDF client shall still use the legacy parser.
@tomuben tomuben self-assigned this Oct 1, 2024
@tomuben tomuben added the refactoring Code improvement without behavior change label Oct 1, 2024
This was referenced Oct 2, 2024
Merged
Merged
tomuben added a commit that referenced this issue Oct 15, 2024
@tomuben
#969: Use new CTPG Parser in Java VM
7a9c654
tomuben added a commit to exasol/script-languages that referenced this issue Oct 15, 2024
@tomuben @tkilias
#969: Use new CTPG parser in java vm (#455)
a2a2f0b 
related to exasol/script-languages-release#969

---------

Co-authored-by: Torsten Kilias <[email protected]>
@tomuben tomuben linked a pull request Oct 16, 2024 that will close this issue
#969: Use new CTPG parser in java vm #970
Merged
10 tasks
tomuben added a commit that referenced this issue Oct 16, 2024
@tomuben
#969: Use new CTPG parser in java vm (#970)
de7d41d 
fixes #969
@tomuben tomuben closed this as completed Oct 16, 2024
tomuben added a commit that referenced this issue Nov 5, 2024
@tomuben
Merge pull request #1000 from exasol/develop
4633316 
Changelist:
- #969: Use new CTPG parser in java vm (#970) 
- #980: Moved flavor template-Exasol-all-java-17 to script-languages repo (#984)
- #971: Removed dead code in Java VM (#985) 
- #982 &  #983: Refactorings in ScriptOptions parser (#987) 
- #988: Adapted Java integration tests for new ScriptOptions parser (#995) 
- #997: Added documentation for Rollback of Script-Languages-Container (#999)
tomuben added a commit that referenced this issue Nov 14, 2024
@tomuben
Update master
f2edb41 
Changelist:

- #944: Removed exaudfclient_bin_py3 (#975) 
- #967: Added ctpg script options parser (#968) 
- #972 and #973: Refactorings for Script Options parser (#976) 
- #977: Fixed Trivy update cache workflow (#978) 
- #969: Use new CTPG parser in java vm (#970) 
- #980: Moved flavor template-Exasol-all-java-17 to script-languages repo (#984)
- #971: Removed dead code in Java VM (#985) 
- Refactorings in ScriptOptions parser (#987) 
- #988: Adapted Java integration tests for new ScriptOptions parser (#995) 
- #997: Added documentation for Rollback of Script-Languages-Container (#999)
- #878: Fixed jvmoption and parameters containing a space (#1001) 
- #1002: Use ECR fallback repository for Trivy caching (#1004) 
- #1006: Updated Dependencies on top of 8.4.0 (#1007) 
- #958: Fix conda version conflict (#1010) 
- #1009: Removed NVIDIA GPU Driver Libraries from template-CUDA SLC (#1012)
- Bumped version to 9.0.0 (#1013) 
- #1014: Fixed Push Docker Test Container workflow (#1015)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomuben tomuben

Labels
refactoring Code improvement without behavior change
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#969: Use new CTPG parser in java vm exasol/script-languages-release
1 participant
@tomuben