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

Remove dead code in Java VM #971

Closed
tomuben opened this issue Oct 8, 2024 · 0 comments · Fixed by #985
Closed

Remove dead code in Java VM #971

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

Comments

@tomuben
Copy link
Collaborator

tomuben commented Oct 8, 2024

Member m_exceptionThrown in class JavaVMImpl is not used and can be removed:
https://github.com/exasol/script-languages/blob/ca29b6f41f107c0c6f9142284b855dd86d3f16d3/exaudfclient/base/javacontainer/javacontainer_impl.h#L46
@tomuben tomuben assigned tomuben and unassigned tomuben Oct 8, 2024
@tomuben tomuben added the refactoring Code improvement without behavior change label Oct 8, 2024
@tomuben tomuben mentioned this issue Oct 15, 2024
Merged
@tomuben tomuben self-assigned this Oct 15, 2024
tomuben added a commit to exasol/script-languages that referenced this issue Oct 17, 2024
@tomuben
#971: Removed dead code in Java VM (#460)
5e098e0 
related to exasol/script-languages-release#971
tomuben added a commit that referenced this issue Oct 17, 2024
@tomuben
#971: Removed dead code in Java VM
aa9741a
@tomuben tomuben mentioned this issue Oct 17, 2024
Merged
10 tasks
@tomuben tomuben linked a pull request Oct 17, 2024 that will close this issue
#971: Removed dead code in Java VM #985
Merged
10 tasks
tomuben added a commit that referenced this issue Oct 17, 2024
@tomuben
#971: Removed dead code in Java VM (#985)
5bd0a7a 
fixes #971
@tomuben tomuben closed this as completed Oct 17, 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.

#971: Removed dead code in Java VM exasol/script-languages-release
1 participant
@tomuben