### What is this PR for?
- ToolProvider.getSystemJavaCompiler() can return null in JRE environments.
- Added a fail-fast null check in StaticRepl to throw a clear exception early.
- Updated unit test to reflect this behavior.
### What type of PR is it?
Bug Fix
### Todos
* [ ] - Task
### What is the Jira issue?
* [ZEPPELIN-6306](https://issues.apache.org/jira/browse/ZEPPELIN-6306)
### How should this be tested?
* Run the existing unit test `testJDKCompilerAvailability()` to ensure the compiler is available in JDK.
* Run all Zeppelin unit tests to confirm no regressions.
### Screenshots (if appropriate)
### Questions:
* Does the license files need to update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Closes#5069 from eunhwa99/ZEPPELIN-6306.
Signed-off-by: ParkGyeongTae <gyeongtae@apache.org>
### What is this PR for?
This PR refactors the `StaticRepl` class to improve readability, maintainability, and align with modern Java best practices.
There are no functional changes.
- Generic type refinement with diamond operator in loop: Adds type safety by explicitly specifying the generic type. Prevents unchecked warnings and makes the code clearer to readers and tools.
- Replace `Arrays.asList` to `List.of` : List.of (Java 9+) produces an immutable list, which is safer and better communicates the intent that the list will not be modified. It avoids side effects of Arrays.asList, which is fixed-size and backed by the original array.
- Minor typos fixed: Corrected spelling mistakes and adjusted comments.
- Removes redundant boolean comparison to follow standard Java coding conventions.
- Replace index-based loop with enhanced for-each to eliminate boilerplate index management, improves readability, and reduces chances of off-by-one errors. It makes the intent ("iterate all elements") clearer.
- Variable declaration moved closer to usage to improve code locality and readability by reducing variable scope. This makes the code easier to maintain and follow.
### What type of PR is it?
Refactoring
### Todos
* [ ] - Task
### What is the Jira issue?
* [ZEPPELIN-6299](https://issues.apache.org/jira/browse/ZEPPELIN-6299)
### How should this be tested?
* Run existing unit tests to confirm behavior remains unchanged.
### Screenshots (if appropriate)
### Questions:
* Does the license files need to update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Closes#5048 from eunhwa99/ZEPPELIN-6299.
Signed-off-by: ParkGyeongTae <gyeongtae@apache.org>
### What is this PR for?
Change the version to 0.13.0-SNAPSHOT
### What type of PR is it?
Improvement
### Todos
* [x] - Change versions to 0.13.0-SNAPSHOT
### How should this be tested?
The version should show 0.13.0-SNAPSHOT when building it.
### Screenshots (if appropriate)
### Questions:
* Does the license files need to update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Closes#4995 from jongyoul/minor/bump-up-0.13.0-snapshot.
Signed-off-by: Jongyoul Lee <jongyoul@gmail.com>
### What is this PR for?
This pull request updates the Apache parent version. We benefit significantly from plugin updates and can clean up our own `pom.xml`.
plugin updates:
maven-antrun-plugin - 1.8 -> 3.1.0
maven-assembly-plugin - 3.7.1 -> 3.7.1
maven-clean-plugin - 3.1.0 -> 3.4.0
maven-compiler-plugin - 3.8.1 -> 3.13.0
maven-dependency-plugin - 3.1.2 -> 3.7.1
maven-deploy-plugin - 2.8.2 -> 3.1.2
maven-enforcer-plugin - 3.0.0-M3 -> 3.5.0
maven-gpg-plugin - 1.6 -> 3.2.4
apache-rat-plugin - 0.13 -> 0.16.1
surefire & failsafe version - 3.2.2 -> 3.3.0
### What type of PR is it?
Improvement
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-6138
### How should this be tested?
* CI
### Questions:
* Does the license files need to update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Closes#4884 from Reamer/apacheParent.
Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
* [ZEPPELIN-6038] Unify the Logger variable to LOGGER in all files
* [ZEPPELIN-6038] Set all logger constants to private static final
* [ZEPPELIN-6038] Adjust code character length
* [ZEPPELIN-6038] Refactor logger variable
* [ZEPPELIN-6038] Revert code for automatically generated files
* [ZEPPELIN-6038] Fix code syntax
### What is this PR for?
This PR removes unnecessary maven-dependency-plugin maven plugin calls
### What type of PR is it?
- Refactoring
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-5522
### How should this be tested?
* CI
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Philipp Dallig <philipp.dallig@gmail.com>
Closes#4224 from Reamer/maven-dependency-plugin and squashes the following commits:
1a73f4f29 [Philipp Dallig] Remove maven-dependency-plugin from zeppelin-distribution
aa5d203d9 [Philipp Dallig] Remove maven-dependency-plugin from zeppelin-interpreter-parent
ff562e246 [Philipp Dallig] Remove plugin maven-dependency-plugin, which is skipped by default in zeppelin-interpreter-parent
43ee00ca7 [Philipp Dallig] Skip maven-dependency-plugin copy-plugin-dependencies for k8s-standard
### What is this PR for?
Version up to the next version of 0.11.0-SNAPSHOT
Author: Jongyoul Lee <jongyoul@gmail.com>
Closes#4221 from jongyoul/minor-version-up and squashes the following commits:
c38c9910a [Jongyoul Lee] Version up to the latest stable version
a962b08ec [Jongyoul Lee] [MINOR] Version up to 0.11.0-SNAPSHOT
### What is this PR for?
Polish pom.xml files
- Delete unnecessary tags
- Delete duplicated dependencies
### What type of PR is it?
[Refactoring]
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-5370
### How should this be tested?
* CI
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: cuspymd <cuspymd@gmail.com>
Closes#4118 from cuspymd/polish-poms and squashes the following commits:
766278338 [cuspymd] Fix integration test error
385fcf2fd [cuspymd] Delete duplicated version of sub-modules
ba127bf16 [cuspymd] Recover duplicated dependencies related to spark interpreter
6fa7e1d51 [cuspymd] Polish pom.xml
This issue is based out of comment https://github.com/apache/zeppelin/pull/3370#issuecomment-511281165, where Injellij shows unknown error.
[Improvement]
* [ZEPPELIN-4244](https://issues.apache.org/jira/browse/ZEPPELIN-4244)
* CI should be green
* Intellij IDE should not show any error
* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A
Author: Prabhjyot Singh <prabhjyot.singh@cloudera.com>
Closes#3409 from prabhjyotsingh/ZEPPELIN-4244 and squashes the following commits:
840ef118b [Prabhjyot Singh] ZEPPELIN-4244: Intellij shows URI not regietered error for xmlns urls
Change-Id: I88afbbb79967648f2932d33524e68fb2c5670006
### What is this PR for?
Make use of a Java stream instead of an ugly for-loop to get rid of class files.
### What type of PR is it?
Improvement
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3880
### How should this be tested?
* CI pass.
### Questions:
* Does the licenses files need update? No.
* Is there breaking changes for older versions? No.
* Does this needs documentation? No.
Author: Patrice Clement <monsieurp@gentoo.org>
Closes#3241 from monsieurp/ZEPPELIN-3880 and squashes the following commits:
e174285e8 [Patrice Clement] ZEPPELIN-3880: Code refactoring: remove class files using a Java stream
### What is this PR for?
This is refactoring PR which move common dependencies into zeppelin-interpreter-parent
### What type of PR is it?
[Refactoring]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3790
### How should this be tested?
* Ci pass
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jeff Zhang <zjffdu@apache.org>
Closes#3194 from zjffdu/ZEPPELIN-3790 and squashes the following commits:
0b34645a0 [Jeff Zhang] move junit to zeppelin-interpreter-parent
e4baa595e [Jeff Zhang] ZEPPELIN-3790. Move common interpreter dependencies into zeppelin-interpreter-parent
### What is this PR for?
Trivial PR for module name refactoring.
### What type of PR is it?
[Refactoring]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3774
### How should this be tested?
* CI pass
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jeff Zhang <zjffdu@apache.org>
Closes#3175 from zjffdu/ZEPPELIN-3774 and squashes the following commits:
ea72df9e5 [Jeff Zhang] ZEPPELIN-3774. Rename module interpreter-parent to zeppelin-interpreter-parent
### What is this PR for?
This PR introduce new module zeppelin-interpreter-api which will shade all dependencies of zeppelin-interpreter, this is just to avoid the conflict with the custom interpreter implementation which may use jars that conflicts with dependency of zeppelin-interpreter.
### What type of PR is it?
[ Improvement]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3689
### How should this be tested?
* CI pass
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jeff Zhang <zjffdu@apache.org>
Closes#3170 from zjffdu/ZEPPELIN-3689 and squashes the following commits:
f3f210aee [Jeff Zhang] ZEPPELIN-3689. Shade all dependencies of zeppelin-interpreter
### What is this PR for?
Avoiding outdated checkstyle error.
To check format
`mvn validate`
To fix non-complying files
`mvn fmt:format`
### What type of PR is it?
[Improvement | Refactoring]
### Todos
* [x] - Adopt `fmt-maven-plugin`
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3740
### How should this be tested?
CI passs
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Yes for contributors
Author: Jongyoul Lee <jongyoul@gmail.com>
Closes#3158 from jongyoul/ZEPPELIN-3740 and squashes the following commits:
f06c50e90 [Jongyoul Lee] Adopt `fmt-maven-plugin` for google-java-format Adopt all files
### What is this PR for?
**New Java interpreter**
There are several Java libraries that could be used to leverage the Data Visualization & Collaboration features of Zeppelin hence the need of a Java interpreter to run any java code / library with no further dependencies.
Dependencies to any java library can be added by the end users in the Java Zeppelin interpreter Dependencies settings section.
### What type of PR is it?
* Feature
### Todos
* Any feedback from reviewers
### What is the Jira issue?
* [ZEPPELIN-3653]
### How should this be tested?
Manually
* Start the Zeppelin server
* Create a new note with the java interpreter binding
* Write some java code as per documentation (docs/interpreter/java.md)
Unit tests
* Run unit tests (JavaInterpreterTest.java and JavaInterpreterUtilsTest.java)
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update?
No, the dependency on com.thoughtworks.qdox was already added as part of the Beam Interpreter.
* Is there breaking changes for older versions?
No.
* Does this needs documentation?
Yes, it has been added to the PR, see docs/interpreter/java.md.
Author: Vincenzo Selvaggio <vselvaggio@hotmail.it>
Closes#3092 from selvinsource/java-interpreter and squashes the following commits:
c5f96b27d [Vincenzo Selvaggio] Excluding java interpreter from zeppelin-server tests.
3f85f1cd3 [Vincenzo Selvaggio] Removed redundant java profile as result of rebasing.
b68634034 [Vincenzo Selvaggio] Updated documentation for Java interpreter to be more explicit on the limitations.
ce9401079 [Vincenzo Selvaggio] Updated Java Interpreter README and documentation.
9e9a859e9 [Vincenzo Selvaggio] Stylecheck fixes.
706aa0795 [Vincenzo Selvaggio] PR reviews changes: -updated interpreter list -direct import of java module and update related README -static logger for Java interpreter
d2358f5dc [Vincenzo Selvaggio] Updated documentation for Java interpreter by adding relevant examples.
b57f1cfd0 [Vincenzo Selvaggio] Added Java Interpreter Utils and tests for them.
f542a4f61 [Vincenzo Selvaggio] Java Interpreter first version
